chore: Add missing form fields to RNA example app #4583
Conversation
|
| updated_at: { | ||
| label: 'Updated at', | ||
| isRequired: false, | ||
| type: 'text', | ||
| }, | ||
| address: { | ||
| label: 'Address', | ||
| isRequired: false, | ||
| type: 'text', | ||
| }, | ||
| gender: { | ||
| label: 'Gender', | ||
| isRequired: false, | ||
| type: 'text', | ||
| }, | ||
| locale: { | ||
| label: 'Locale', | ||
| isRequired: false, | ||
| type: 'text', | ||
| }, |
There was a problem hiding this comment.
Not sure how much value including these in the example code provides, imo kind of bloats the example code. Are the keys not documented?
There was a problem hiding this comment.
Cognito is configured with these attributes as required, and once created we can't modify the user pool attributes. The value is that we can perform manual tests with this app.
There was a problem hiding this comment.
Is this example used in the docs?
There was a problem hiding this comment.
No, docs use this example
ioanabrooks
changed the base branch from
next-release/migration
to
next-release/main
ioanabrooks
dismissed
dbanksdesign’s stale review
The base branch was changed.
ioanabrooks
changed the base branch from
next-release/main
to
next-release/merge-main-to-migration
reesscot
merged commit bb65e5b
into
next-release/merge-main-to-migration
Description of changes
Add missing fields from signup with attributes RNA example app.
Issue #, if available
Description of how you validated changes
Checklist
yarn testpasses and tests are updated/addedsideEffectsfield updatedBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.