Skip to content

Add display order compound fields#12040

Merged
sekmiller merged 1 commit intodevelopfrom
compound-field-ordering
Dec 12, 2025
Merged

Add display order compound fields#12040
sekmiller merged 1 commit intodevelopfrom
compound-field-ordering

Conversation

@scolapasta
Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
Adds ordering to the getter for compound values
Which issue(s) this PR closes:

Special notes for your reviewer:
one line fix

Suggestions on how to test this:
if you can create a datset where the ordering is wrong before testing; then apply fix and confirm ordering is fixed

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
no (besides a fix to the order)

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

Additional documentation:

@github-actions github-actions Bot added GREI Re-arch Issues related to the GREI Dataverse rearchitecture Size: 3 A percentage of a sprint. 2.1 hours. SPA These changes are required for the Dataverse SPA Type: Bug a defect labels Dec 11, 2025
@scolapasta scolapasta added this to the 6.9 milestone Dec 11, 2025
@scolapasta scolapasta moved this to Ready for Review ⏩ in IQSS Dataverse Project Dec 11, 2025
Copy link
Copy Markdown
Member

@qqmyers qqmyers left a comment

Choose a reason for hiding this comment

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

Looks good - haven't tested.

@github-project-automation github-project-automation Bot moved this from Ready for Review ⏩ to Reviewed but Frozen ❄️ in IQSS Dataverse Project Dec 11, 2025
@scolapasta scolapasta moved this from Reviewed but Frozen ❄️ to Ready for QA ⏩ in IQSS Dataverse Project Dec 11, 2025
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 24.231%. remained the same
when pulling bdbc7b8 on compound-field-ordering
into 70b6e26 on develop.

@sekmiller sekmiller self-assigned this Dec 12, 2025
@sekmiller sekmiller moved this from Ready for QA ⏩ to QA ✅ in IQSS Dataverse Project Dec 12, 2025
@github-actions
Copy link
Copy Markdown

📦 Pushed preview images as

ghcr.io/gdcc/dataverse:compound-field-ordering
ghcr.io/gdcc/configbaker:compound-field-ordering

🚢 See on GHCR. Use by referencing with full name as printed above, mind the registry name.

@sekmiller sekmiller merged commit fd75558 into develop Dec 12, 2025
26 of 28 checks passed
@github-project-automation github-project-automation Bot moved this from QA ✅ to Merged 🚀 in IQSS Dataverse Project Dec 12, 2025
@scolapasta scolapasta moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project Dec 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GREI Re-arch Issues related to the GREI Dataverse rearchitecture Size: 3 A percentage of a sprint. 2.1 hours. SPA These changes are required for the Dataverse SPA Type: Bug a defect

Projects

Status: Done 🧹

Development

Successfully merging this pull request may close these issues.

Display Order not being returned correctly for Dataset Authors

4 participants