Skip to content
This repository has been archived by the owner on Apr 29, 2022. It is now read-only.

Extension bug: When asked to login on OSrPRE via extension it redirects to website #138

Open
dasaderi opened this issue Jul 7, 2020 · 15 comments
Labels
bug Something isn't working 🎁 Rewarded on Issuehunt This issue has been rewarded on Issuehunt Mozilla 2020 Sprints question Further information is requested

Comments

@dasaderi
Copy link
Member

dasaderi commented Jul 7, 2020

Issuehunt badges

On Firefox (I have not verified this on Chrome) when using the extension to add a review directly from a preprint server the site asks to login if not yet logged. But the problem is that once orcid-mediated login happens the site brings user to the platform rather than to the form on the extension as it should do.


IssueHunt Summary

leonardosfl leonardosfl has been rewarded.

Backers (Total: $100.00)

Submitted pull Requests


Tips

@dasaderi dasaderi added the bug Something isn't working label Jul 7, 2020
@murkatr
Copy link

murkatr commented Jul 7, 2020

It does this same thing on Chrome.

@issuehunt-oss
Copy link

issuehunt-oss bot commented Jul 7, 2020

@prereview has funded $100.00 to this issue.


@issuehunt-oss issuehunt-oss bot added the 💵 Funded on Issuehunt This issue has been funded on Issuehunt label Jul 7, 2020
@leonardosfl
Copy link
Contributor

Hi, @dasaderi! I've implemented a fix for this. I just wanted to know if this is the right behavior.

It redirects to the login page and then returns to the previous page and opens the extension form.

Login modal:
login-popup

Login button:
login

@dasaderi
Copy link
Member Author

@leonardosfl Thank you so much! Great work!

leonardosfl added a commit to leonardosfl/rapid-prereview that referenced this issue Jul 18, 2020
@leonardosfl
Copy link
Contributor

@dasaderi Thanks! I've submitted the PR.

@jheretic
Copy link
Contributor

One more point of clarification on user interaction, @dasaderi:

@leonardosfl's #145 does fix the issue as shown. But I just wanted to mention that if there are no existing reviews or requests, it seems as if you can only login via the login link in the dropdown menu (since there's no way to summon the drawer). If you login via that link, it ultimately brings you to https://oubreaksci.prereview.org, but in a new tab. The original tab still exists and is now logged in. I think this is an acceptable workflow, I just wanted to make sure this reflects your expectations.

@leonardosfl
Copy link
Contributor

it seems as if you can only login via the login link in the dropdown menu (since there's no way to summon the drawer)

It's possible to summon the drawer by pulling it from the bottom.

drawer

If you login via that link, it ultimately brings you to https://oubreaksci.prereview.org

I was not sure if I should change that link too, but it should be possible.

harumhelmy added a commit that referenced this issue Jul 27, 2020
@leonardosfl
Copy link
Contributor

Hi @dasaderi, Is this fixed?

@jheretic
Copy link
Contributor

Hi @leonardosfl, I'm marking this as fixed.

@issuehunt-oss
Copy link

issuehunt-oss bot commented Jul 28, 2020

@murkatr has rewarded $90.00 to @leonardosfl. See it on IssueHunt

  • 💰 Total deposit: $100.00
  • 🎉 Repository reward(0%): $0.00
  • 🔧 Service fee(10%): $10.00

@issuehunt-oss issuehunt-oss bot added 🎁 Rewarded on Issuehunt This issue has been rewarded on Issuehunt and removed 💵 Funded on Issuehunt This issue has been funded on Issuehunt labels Jul 28, 2020
@dasaderi dasaderi reopened this Aug 19, 2020
@dasaderi
Copy link
Member Author

@leonardosfl @jheretic I just logged in from the extension in Chrome and it did it again. After I clicked on "Log in with your Orcid", screen that showed after I tried to Add a review from medRxiv

Screen Shot 2020-08-19 at 3 12 09 PM

a new window opened for the login and then I landed again in the website homepage. Has this change been deployed?

@dasaderi
Copy link
Member Author

Additionally, even if I'm logged in, when I go back to the window on meRxiv, it still does not detect me as logged in.

@leonardosfl
Copy link
Contributor

@dasaderi I think it has not been deployed yet
ext

@dasaderi
Copy link
Member Author

You are right, @leonardosfl. My mistake. :D We are on it!

@dasaderi
Copy link
Member Author

@jheretic was this deployed?

@rudietuesdays rudietuesdays added the question Further information is requested label Oct 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working 🎁 Rewarded on Issuehunt This issue has been rewarded on Issuehunt Mozilla 2020 Sprints question Further information is requested
Projects
None yet
Development

No branches or pull requests

5 participants