Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

feat(app-data): upgrade to latest appData version 1.1.0 #651

Merged
merged 7 commits into from
Nov 1, 2023

Conversation

alfetopito
Copy link
Collaborator

@alfetopito alfetopito commented Oct 19, 2023

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!
image

Summary

Upgrade to latest appData version 1.1.0

It contains the new metadata fields: hooks and signer

image

image

To Test

  1. From the menu, navigate to the appData tab:
    image
  2. You should see the new metadata fields on the left side form.

Notes

  1. CSS seems a bit broken, I can't investigate this now
  2. There might be more work required, I also have not tried this too hard

@vercel
Copy link

vercel bot commented Oct 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
explorer-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2023 6:09pm

@alfetopito alfetopito requested a review from a team October 19, 2023 12:47
@github-actions
Copy link

Pull Request Test Coverage Report for Build 6574800324

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 61.711%

Totals Coverage Status
Change from base Build 6532212693: 0.0%
Covered Lines: 878
Relevant Lines: 1237

💛 - Coveralls

@fedgiac
Copy link

fedgiac commented Oct 19, 2023

The buttons for the hooks look weird to me. Shouldn't they have some text on them, like "add" or "delete"?

@socket-security
Copy link

socket-security bot commented Oct 19, 2023

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@cowprotocol/app-data 1.0.2...1.1.0 None +0/-0 175 kB cowprotocol_dev

@alfetopito
Copy link
Collaborator Author

The buttons for the hooks look weird to me. Shouldn't they have some text on them, like "add" or "delete"?

That's what I mean in the notes section 😅
Some CSS work might be required, which I have no time nor skill to address right now.

Copy link
Contributor

@anxolin anxolin left a 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!

@alfetopito
Copy link
Collaborator Author

Great that it worked automatically, this used to be a PAIN!

Well, kind of.
The CSS needs some love, as Federico pointed out.

@alfetopito alfetopito force-pushed the upgrade-to-latest-app-data branch from 986a6fb to 4b9157c Compare October 31, 2023 18:05
Comment on lines +122 to +140
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';
}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is it

Copy link
Contributor

@shoom3301 shoom3301 left a 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?)

@alfetopito alfetopito merged commit bb5f08a into develop Nov 1, 2023
7 checks passed
@alfetopito alfetopito deleted the upgrade-to-latest-app-data branch November 1, 2023 11:13
alfetopito added a commit that referenced this pull request Nov 1, 2023
* 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
alfetopito added a commit that referenced this pull request Nov 1, 2023
* 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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants