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.

Sign up for GitHub

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

Jump to bottom

Plugin writes an empty https://[sandbox].orcid.org by failed communication #214

Closed
withanage opened this issue Aug 3, 2022 · 0 comments
Closed

Plugin writes an empty https://[sandbox].orcid.org by failed communication #214

withanage opened this issue Aug 3, 2022 · 0 comments
Labels
bug
Milestone
1.1.3.3

Comments

@withanage
Copy link
Member

Describe the bug

Plugin writes an empty https://[sandbox].orcid.org by failed communication

To Reproduce
Difficult scenario to recreate on API status, network or other third party or unkown factors.

Expected behavior

  • Clear and consise message, what has happened
  • Do not write the orcid url without the id into the database
@withanage withanage added the bug label Aug 3, 2022
@withanage withanage added this to the 1.1.3.3 milestone Aug 3, 2022
withanage added a commit to withanage/orcidProfile that referenced this issue Aug 3, 2022
@withanage
pkp#214 Allow valid orcid ids into database, refactor API errors
578ae76
withanage added a commit to withanage/orcidProfile that referenced this issue Aug 5, 2022
@withanage
pkp#214 Allow valid orcid ids into database, refactor API errors
75dac64
@withanage withanage mentioned this issue Aug 5, 2022
Merged
withanage added a commit that referenced this issue Aug 7, 2022
@withanage
Merge pull request #215 from withanage/stable-3_3_0
ff62b0c 
#214  Allow valid orcid ids into database, refactor API Errors
@withanage withanage mentioned this issue Aug 7, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
1.1.3.3
Development

No branches or pull requests
1 participant
@withanage