Skip to content

[ANG-9250] P3.5 - Some registration schema fields not displaying or formatting correctly in Angular UI#694

Closed
nsemets wants to merge 31 commits into
CenterForOpenScience:feature/pbs-25.02from
nsemets:fix/ANG-9250
Closed

[ANG-9250] P3.5 - Some registration schema fields not displaying or formatting correctly in Angular UI#694
nsemets wants to merge 31 commits into
CenterForOpenScience:feature/pbs-25.02from
nsemets:fix/ANG-9250

Conversation

@nsemets

@nsemets nsemets commented Oct 22, 2025

Copy link
Copy Markdown
Collaborator
  • Ticket: [ANG-9250]
  • Feature flag: n/a

Summary of Changes

  1. Fixed display of schema blocks.

nsemets and others added 30 commits October 20, 2025 11:51
[ENG-9224] Add contributors in preprint submission
[ENG-9257] Reorder metadata on preprint detail page for visibility and consistency
[ENG-9250] Some registration schema fields not displaying or formatting correctly in Angular UI
[ENG-9224] Bibliographic author list should be displayed in reviews app, not just the creator
[ENG-9613] Submitting new versions to OSF Preprints no longer publishes new version
[ENG-9235] Logo on preprint, registries, collections, and institutions landing/discovery page is too small
…ix/legacy_wiki_redirect

[ENG-9249] Legacy wiki redirect
[ENG-9230] P6.4 - Visible components on project overview page is limited to 10. All direct children need to be available on the overview page.
@brianjgeiger

Copy link
Copy Markdown
Contributor

We'll be changing the branch to feature/pbs-25.02 once it's finished pushing to centerforopenscience.

@nsemets nsemets changed the base branch from feature/pbs-25.01 to feature/pbs-25.02 October 22, 2025 14:23
@nsemets nsemets changed the base branch from feature/pbs-25.02 to feature/pbs-25.01 October 22, 2025 14:23
@brianjgeiger

Copy link
Copy Markdown
Contributor

@nsemets Please rebase on feature/pbs-25.02

Comment on lines +36 to 43
@if (page.questions?.length) {
<osf-registration-blocks-data
[questions]="page.questions"
[reviewData]="schemaResponseRevisionData()"
></osf-registration-blocks-data>
}

@if (page.sections?.length) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Note for posterity, this addresses an issue where certain questions were not showing up in the overview page. This was stemming from the fact that there was an implicit assumption in how we were rendering forms before where we assumed the blocks would show up in an order like PageHeading -> SectionHeading -> QuestionLabel, but that is not necessarily the case

@nsemets nsemets changed the base branch from feature/pbs-25.01 to feature/pbs-25.02 October 22, 2025 14:28
@nsemets nsemets requested a review from futa-ikeda October 22, 2025 14:31

@futa-ikeda futa-ikeda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Just a couple places where we may be able to avoid similar issues in the future

Comment thread src/app/shared/mappers/registration/page-schema.mapper.ts Outdated
Comment thread src/app/shared/mappers/registration/page-schema.mapper.ts Outdated
@futa-ikeda

Copy link
Copy Markdown
Contributor

Leaving a note here to describe the problem for ourselves in the future. The issue that was appearing was that certain sections of the registration template would not appear in the submission and the overview page

Missing sections on the submission page:

  • This issue would occur when multiple, back-to-back paragraph block types were in a template. The subsequent paragraph blocks would override the contents of the previous paragraph block that was stored in currentSection.description. The fix is to append the paragraph content to the existing description

Missing sections in the review and overview pages:

  • This issue was caused by the fact that there was a subtle assumption being made when rendering that schemas would follow the pattern of PageHeading -> SectionHeading -> QuestionLabel, but schemas may just go PageHeading -> QuestionLabel directly (skipping SectionHeading)

@nsemets nsemets changed the title fix(registration-schemas): fixed schemas blocks display [ANG-9250] P3.5 - Some registration schema fields not displaying or formatting correctly in Angular UI Oct 22, 2025
@nsemets nsemets closed this Oct 22, 2025
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.

4 participants