forked from ORCID/ORCID-Source
-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] main from ORCID:main #808
Open
pull
wants to merge
1,076
commits into
NOUIY:main
Choose a base branch
from
ORCID:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Deactivated records should get 409 on GET requests * Add better error message for when a issn couldnt be found * Running locally
* Deactivated records should get 409 on GET requests * Add better error message for when a issn couldnt be found * Running locally * Banned exception and more logging
* Deactivated records should get 409 on GET requests * Reset fail count when it is updated
* do not send email change notifications * adjust tests * adjust another test * remove more instances of notification being sent * remove all instances of the sendEmailAddressChangedNotification function --------- Co-authored-by: Angel Montenegro <[email protected]>
* add timestamp to date pojo * remove duplicated and unused code
* Deactivated records should get 409 on GET requests * On OBO, When a token is user disabled, it should be possible to use the short lived token to delete * More unit tests
Co-authored-by: Angel Montenegro <[email protected]>
Co-authored-by: Angel Montenegro <[email protected]>
* Deactivated records should get 409 on GET requests * Do not cache the token if it id disabled, since the token does not know its disabled state * Remove unused imports
* fix typos * add db table * add togglz * add email domain logic * improve new domain logic * add another toggle * improve readability, add transactions, cleanup * remove todo * add indexes to profile email domain table * fix new domain visibility bug
* Deactivated records should get 409 on GET requests * Sequence should start at 100000
* Update the csrfTokenRepo bean so we can configure the cookie domain * Use the factory method to set the httpcookie setting
Fetch only the org id in the initial search
…-it-doesnt-use-the-db-so-often
…ta-logic-so-it-doesnt-use-the-db-so-often RateLimitig with Redis Implementation
Added the check for existent entries
…sage for DB sync. (#7190)
* Added the translation for the papi quota email * Update MailGunManager.java Removed the code commited by mistake * development.properties removed QA env from props * Update MailGunManager.java * Update development.properties * Remove unused param --------- Co-authored-by: amontenegro <[email protected]>
Co-authored-by: Angel Montenegro <[email protected]>
Add affiliation after reactivation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )