-
Notifications
You must be signed in to change notification settings - Fork 8
feat(app-data): upgrade to latest appData version 1.1.0 #651
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Pull Request Test Coverage Report for Build 6574800324
💛 - Coveralls |
The buttons for the hooks look weird to me. Shouldn't they have some text on them, like "add" or "delete"? |
Updated dependencies detected. Learn more about Socket for GitHub ↗︎
|
That's what I mean in the notes section 😅 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great that it worked automatically, this used to be a PAIN!
Well, kind of. |
986a6fb
to
4b9157c
Compare
i.glyphicon { | ||
display: none; | ||
} | ||
.btn-add::after { | ||
content: 'Add'; | ||
} | ||
.array-item-copy::after { | ||
content: 'Copy'; | ||
} | ||
.array-item-move-up::after { | ||
content: 'Move Up'; | ||
} | ||
.array-item-move-down::after { | ||
content: 'Move Down'; | ||
} | ||
.array-item-remove::after { | ||
content: 'Remove'; | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
The only nitpick about commented code (should we delete it?)
* feat(app-data): upgrade to latest appData version 1.1.0 * chore: show appData array buttons * chore: comment out ipfs stuff * refactor: remove stuff that's not needed and remove some `any`s * refactor: sort imports * chore: remove IPFS reference * refactor: format
* feat(app-data): upgrade to latest appData version 1.1.0 * chore: show appData array buttons * chore: comment out ipfs stuff * refactor: remove stuff that's not needed and remove some `any`s * refactor: sort imports * chore: remove IPFS reference * refactor: format
Update 2023-10-31
After lots of digging, found out what I needed to update: https://rjsf-team.github.io/react-jsonschema-form/docs/advanced-customization/internals/#custom-array-field-buttons
Now we have add, remove, move up/down buttons!
Summary
Upgrade to latest appData version 1.1.0
It contains the new metadata fields:
hooks
andsigner
To Test
Notes