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

ASAP-686 CRN Public API changes #4408

Merged
merged 2 commits into from
Oct 8, 2024
Merged

Conversation

gabiayako
Copy link
Collaborator

@gabiayako gabiayako commented Oct 7, 2024

JIRA ticket: https://asaphub.atlassian.net/browse/ASAP-686


This part of acceptance

Confirm Research Theme field is removed from Users and Research Outputs in CMS

Will be done in another PR as ideally it requires a new Contentful environment


Examples:

https://api-4408.hub.asap.science/public/research-outputs/1QDoTyaLa25aZJNIkGz3RH
https://api-4408.hub.asap.science/public/users/edca36c4-ce51-4937-a892-5e4222954239

Copy link

codecov bot commented Oct 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.46%. Comparing base (073a027) to head (ed9f833).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4408   +/-   ##
=======================================
  Coverage   99.46%   99.46%           
=======================================
  Files        1054     1054           
  Lines       20372    20374    +2     
  Branches     6431     6429    -2     
=======================================
+ Hits        20263    20265    +2     
  Misses        109      109           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gabiayako gabiayako merged commit 043f236 into master Oct 8, 2024
63 checks passed
@gabiayako gabiayako deleted the ASAP-686-crn-public-api-changes branch October 8, 2024 13:07
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.

2 participants