Skip to content

Conversation

@wshahn
Copy link

@wshahn wshahn commented May 27, 2025

What this PR does / why we need it:
Updates storage quota http method to POST so that pushing storage quotas works as intended.

Which issue(s) this PR closes:
Closes #11530

Special notes for your reviewer:
None.

Suggestions on how to test this:
Issue the api call for updating storage quotas as shown in the docs (https://guides.dataverse.org/en/latest/api/native-api.html#id2). It should fail at PUT indicating the "API endpoint does not support this method."

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No.

Is there a release notes update needed for this change?:
No.

Additional documentation:
None.

@pdurbin pdurbin moved this to Ready for Triage in IQSS Dataverse Project May 27, 2025
Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

I looked at the current code as well as #10144 when the feature was added and it should, indeed, be POST. Thanks, @wshahn! Approved!

@github-project-automation github-project-automation bot moved this from Ready for Triage to Ready for QA ⏩ in IQSS Dataverse Project May 27, 2025
@ofahimIQSS ofahimIQSS self-assigned this May 27, 2025
@ofahimIQSS ofahimIQSS moved this from Ready for QA ⏩ to QA ✅ in IQSS Dataverse Project May 27, 2025
@ofahimIQSS
Copy link
Contributor

Good to go here - merging.

@ofahimIQSS ofahimIQSS merged commit 1d7ea40 into IQSS:develop May 27, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this from QA ✅ to Merged 🚀 in IQSS Dataverse Project May 27, 2025
@ofahimIQSS ofahimIQSS removed their assignment May 27, 2025
@pdurbin pdurbin added this to the 6.7 milestone May 28, 2025
@pdurbin pdurbin moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done 🧹

Development

Successfully merging this pull request may close these issues.

Docs Update: Correct Storage Quota Documentation

3 participants