Skip to content

Review summary - #157

Open
samshao wants to merge 31 commits into
devfrom
review-summary
Open

Review summary#157
samshao wants to merge 31 commits into
devfrom
review-summary

Conversation

@samshao

@samshao samshao commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Authoring dialog properties (set on the component node):
- excludedFields : String[] — field names to suppress (e.g. hidden tokens)
- showEditLinks : Boolean — whether to render "Edit" nav links per section

doris-sun and others added 26 commits April 16, 2026 08:47
eServices - First batch of changes for Petition (form 2) (production)
 eServices - Replicate data from repeatable panels to single text field (production)
…e-panels

eServices - Adapt repeatable helpers client library to make it callable from AEM (production)
eServices - Add mechanism for overriding behaviour of accordion delete button (production)
eServices - Functions for repeatable signature (production)
…le-field

eServices - Re-enable focus on first fillable field (production)
eServices - Navigation fixes (production)
eServices - Fix accordion tabs interoperability with keyboard (production)
…r-helpers

eServices - Helper functions and focus after scribble signatures (production)
eServices - Dynamic signatures (production)
eServices - Canada Post API address component

@heldersrvio heldersrvio left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  • Formatting broken for multiline fields
  • The review page needs to look just like the review page in our forms now. Please create copies of our most recent forms (e.g. HSS/YHCIP and HSS/Children Dental Program/Enroll your child in the Yukon Children's Dental Program) and make sure the layout and styling matches that
  • Remove unrelated code from the PR, such as the AdobeSign service and other client libraries

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.

3 participants