Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Connect button missing on register page #294

Closed
juanCure opened this issue Oct 26, 2023 · 15 comments
Closed

Connect button missing on register page #294

juanCure opened this issue Oct 26, 2023 · 15 comments
Assignees
Milestone

Comments

@juanCure
Copy link

Describe the bug
I have installed a OJS v3.3.0.15 and upgraded the orcidProfile plugin to v1.1.3.10 and set up the plugin with the Sandbox Public API, and I have noticed that the Create or Connect your ORCID ID button is missing on the register a new user's account page. The doubt here is, this is either a expected feature or a bug?

To Reproduce
Steps to reproduce the behavior:

  1. Install OJS v3.3.0.15
  2. Upgrade the orcidProfile module from plugin gallery
  3. Set up the orcidProfile module to use the Sandbox Public API
  4. Go to Register page
    10 register_page

In another way, I have noticed this button appears in the Profile > Public page.
11 profile_page

The enviroment of the server is:
OS: Centos 7
PHP v7.4.33
MySQL 8.0.23

@asmecher
Copy link
Member

@ewhanson ewhanson self-assigned this Oct 26, 2023
ewhanson added a commit to ewhanson/orcidProfile that referenced this issue Oct 27, 2023
@ewhanson
Copy link

ewhanson commented Oct 27, 2023

@asmecher could you have a look? It was a quick fix to add it back in. Not an issue in stable-3_4_0. Thanks!

PR:

@ewhanson
Copy link

Thanks for the report @juanCure! Working on getting a fix for this in.

@asmecher
Copy link
Member

@ewhanson, the PR link above is actually back to this same issue! Can you correct?

@ewhanson
Copy link

@ewhanson, the PR link above is actually back to this same issue! Can you correct?

Oops! Should be correct above now. Thanks!

asmecher added a commit that referenced this issue Nov 14, 2023
#294 Connect button missing on register page
@asmecher
Copy link
Member

Merged, thanks! I'm guessing this needs forward-porting too, right?

@ewhanson
Copy link

Merged, thanks! I'm guessing this needs forward-porting too, right?

Thanks! Nope, it shouldn't. I double checked in stable_3_4_0 and main and the button was present. It seems like it was accidentally removed in stable_3_3_0 at some point.

@asmecher
Copy link
Member

Perfect, thanks!

@jnugent
Copy link
Member

jnugent commented Nov 24, 2023

Hi everyone, with the re-addition of the connect button on the register page we are seeing a strange google recaptcha error now, with some debugging printed into the console. This comes back from Google (this locale key is not in the OJS codebase I don't think)

##common.captcha.error.timeout-or-duplicate##
The form could not be submitted. You may have been logged out. Please reload the page and try again.

and then nothing happens. We get this in the browser JS console:

ORCID Logged In: false [register:177:25]

If this is indeed broken, the feature should probably come back out. Dulip mentioned that it doesn't do much anyway.

ewhanson added a commit to ewhanson/orcidProfile that referenced this issue Nov 24, 2023
@ewhanson
Copy link

Hey @jnugent, it does seem like it was causing an issue. I've gone ahead and undone the change (PR: #303).

@bozana
Copy link

bozana commented Nov 28, 2023

@jnugent, oh, great that you have reported it -- there was a user in the forum having problems with google re-captcha, when installing the new orcid version...
But, how comes actually that this makes a problem? Any idea?
Anyhow, yes, I also think we should then remove it...

@jnugent
Copy link
Member

jnugent commented Nov 28, 2023

Hi @bozana I am not entirely sure, but could it be possible that when the ORCID modal window closes after authenticating, there is a form trigger that is submitting the google recaptcha on the registration form? Then it happens again when the form is actually submitted by the user?

Erik has already reversed his commit so the register via orcid feature is removed now.

@bozana
Copy link

bozana commented Nov 28, 2023

Hi @jnugent, do you maybe also have an 3.4 installation together with re-captcha? The ORCID button is also in 3.4, I think...

@SupremeDimeji
Copy link

Hi @ewhanson @asmecher,
Is there any solution to the missing Connect ID button? Or is it permanently removed now?

I'm having the same issue on ojs 3.3. orcid plugin v1_1_3-12
Thanks

@ewhanson
Copy link

ewhanson commented Jan 5, 2024

Hi @SupremeDimeji, the Connect ID button has not permanently been removed, but there are a few issues that need to be addressed before we add the button back in. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants