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

Feature/gravy valet integration #10665

Conversation

jwalz
Copy link
Collaborator

@jwalz jwalz commented Jul 12, 2024

Purpose

Changes

QA Notes

Please make verification statements inspired by your code and what your code touches.

  • Verify
  • Verify

What are the areas of risk?

Any concerns/considerations/questions that development raised?

Documentation

Side Effects

Ticket

mfraezz and others added 18 commits July 1, 2024 12:08
* Add original_publication_citation to preprints model and serializer

* Fix an issue where original_publication_citation was not updated

* Add unit tests

* Respond to CR:

* Rename original_publication_citation to custom_publication_citation
* Redo migration

* Redo migrations

* Update preprints to route to EOW

* Remvoe the use of preprints_dir

* Add self link and view for preprint subjects relationship (CenterForOpenScience#10619)

* add self link and view for preprint subjects relationship

* Fix permissions issue

* Add required write scopes

---------

Co-authored-by: Brian J. Geiger <bgeiger@pobox.com>

* Get root_folder for preprints (CenterForOpenScience#10630)

## Purpose

Make it so preprints api serialization has a root_folder

## Changes

1. Modify the view to get the root_folder in the case of preprints

## Side Effects

Should only affect preprints

## Ticket

https://openscience.atlassian.net/browse/ENG-5716

---------

Co-authored-by: Yuhuai Liu <yuhuai@cos.io>
Co-authored-by: Brian J. Geiger <bgeiger@pobox.com>
Co-authored-by: Brian J. Geiger <bgeiger@cos.io>
…rOpenScience#10650)

* Prevent users from accessing files from withdrawn registrations via API and Waterbutler

---------

Co-authored-by: Uditi Mehta <uditimehta@coss-mbp.lan>
Co-authored-by: Uditi Mehta <uditimehta@COSs-MacBook-Pro.local>
- Do not record download metrics for renders
@jwalz jwalz force-pushed the feature/gravy_valet_integration branch from 8ca143f to dfaafe0 Compare July 15, 2024 14:25
jwalz and others added 3 commits July 15, 2024 13:25
…OpenScience#10652)

* Get addon data for external providers from GravyValet (if flag is active)
* Further improve `get_auth`

---------

Co-authored-by: Jon Walz <walzj@COSs-MBP.lan>
@jwalz jwalz force-pushed the feature/gravy_valet_integration branch from dfaafe0 to 7c24977 Compare July 15, 2024 17:32
@jwalz jwalz deleted the branch CenterForOpenScience:feature/gravy_valet_integration July 16, 2024 15:24
@jwalz jwalz closed this Jul 16, 2024
@jwalz jwalz deleted the feature/gravy_valet_integration branch July 16, 2024 15:24
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

Successfully merging this pull request may close these issues.

None yet

5 participants