[Reference PR] develop -> IDS/DAZ#11818
Closed
cslzchen wants to merge 16 commits into
Closed
Conversation
…hed (#11787) https://openscience.atlassian.net/browse/ENG-10365 Implement solution from analysis ticket. Here is the issue description and proposed solution: https://openscience.atlassian.net/browse/ENG-10851?focusedCommentId=121306 Call `archive_callback` only after all addons are archived
…eate-version bug (#11788) add admin tool to recover preprints deleted by create-version bug
…ons (#11786) ## Ticket https://openscience.atlassian.net/browse/ENG-11457 ## Purpose Draft registrations and usual registration should have at least 1 subject ## Changes Added validation
* add required_metadata_template (#11680) * add required_metadata_template to API * Fix tests * removed discovery legacy search * removed /search/ legacy route * remove api/v1/user/search/ legacy route * removed api/v1/search/node/ legacy route * removed api/v1/share/search/ legacy route * removed api/v1/search/projects/ legacy route * removed api/v1/search/ and api/v1/search/<type>/ legacy routes * removed unused website/search process_project_search_results function * removed api/v2/search/ endpoint * removed api/v2/search/components/ endpoint * removed api/v2/search/files/ endpoint * removed api/v2/search/projects/ endpoint * removed api/v2/search/registrations/ endpoint * removed api/v2/search/users/ endpoint * removed api/v2/search/institutions/ endpoint * removed api/v2/search/collections/ endpoint, scopes and its dependencies * removed api/v2/search/ base route * removed test related to legacy /explore/ route * Add required metadata validation * Add Required metadata template field to admin pages * fix tests * add field, change admin form * add migration * remove unused field * Update imports * Add switch to enforce required_metadata_template * added cedar metadata record creation/deletion in share for collection submission * removed validate_required_metadata redefinition * enqueue cedar record update in share * added focus_iri to shtrove POST request * separate cedar record identifier to avoid potential issues * separate cedar record serialization * added retry, moved tasks to task__update_share * added more tests, make delete cedar record function working without cedar record existence * fixed tests * simplified turtle test and task__update_share * use delay instead of signature for cedar tasks * fixed migrations conflicts + share conflicts * removed missed test case * removed duplicate method * revert osf/models/provider.py --------- Co-authored-by: Vlad0n20 <137097005+Vlad0n20@users.noreply.github.com> Co-authored-by: Vlad0n20 <vlad.onischuk1234@gmail.com> Co-authored-by: Yuhuai Liu <yuhuai@cos.io>
## Ticket https://openscience.atlassian.net/browse/ENG-10365 ## Purpose `after_register` signal calls `handlers.enqueue_task` on a list of `archive` tasks that call `archive_node` which archives a node and calls waterbutler callback synchronously within asynchronous context. Instead of calling archiving addons and waterbutler callback, return chain of tasks that will be called by `after_register`
…#11769) * Fix missing registries/registrations from Internet Archive backups * Remove unneeded .distinct() from find_ia_backlog
…ations aren't deleted after submission (#11807)
## Ticket https://openscience.atlassian.net/browse/ENG-10340 ## Purpose Fix the user's account getting disabled ## Changes Only block unregistered contributors from having domain names in their full_name.
## Ticket https://openscience.atlassian.net/browse/ENG-10340 ## Purpose Fix notable domains check ## Changes Use lower-case for comparisons
[TBD] PBS-26-13 Project PR
PB&S and ES2 code cleanup
Collaborator
Author
|
The only failure is one linting error from |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
Reference PR