Skip to content
This repository has been archived by the owner on Nov 21, 2024. It is now read-only.

set-schema: fix #70

Merged
merged 1 commit into from
Aug 9, 2024
Merged

set-schema: fix #70

merged 1 commit into from
Aug 9, 2024

Conversation

DonHaul
Copy link
Contributor

@DonHaul DonHaul commented Aug 9, 2024

@@ -116,11 +116,15 @@ def create(self, request):
data=serializer.validated_data["data"],
workflow_type=serializer.validated_data["workflow_type"],
)
workflow["data"][
"$schema":"https://inspirehep.net/schemas/records/authors.json"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's make it config variable, just in case we would like to change it in the future

@DonHaul DonHaul force-pushed the set-schema-fix branch 2 times, most recently from fad6152 to 9091159 Compare August 9, 2024 12:05
@DonHaul DonHaul closed this Aug 9, 2024
@DonHaul DonHaul reopened this Aug 9, 2024
@DonHaul DonHaul self-assigned this Aug 9, 2024
@DonHaul DonHaul marked this pull request as ready for review August 9, 2024 12:10
@DonHaul DonHaul merged commit a202b4d into inspirehep:main Aug 9, 2024
12 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

set_schema overwritting user data coming from inspirehep
2 participants