Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(dfx-orbit): dfx-orbit version 0.5.0 #370

Open
wants to merge 121 commits into
base: main
Choose a base branch
from
Open

feat(dfx-orbit): dfx-orbit version 0.5.0 #370

wants to merge 121 commits into from

Conversation

Sawchord
Copy link
Collaborator

@Sawchord Sawchord commented Oct 2, 2024 •
edited
Loading

This PR introduces dfx-orbit version 0.5.0.
The changes to the last version are quite significant.

Features for asset management:

  • Improved UX to request asset canister permissions
  • Revoke asset canister permissions
  • Commit / Cancel commit
  • Verify commands to match each request

Bug fixes

  • Fix asset verification reproducibility issue when using machines with different CPU architecture

Other changes

  • Tests for asset management
  • Update dependencies

Sawchord and others added 30 commits July 31, 2024 16:56
@Sawchord
Fix test issue
1216364
@Sawchord
Update dependencies
d6ee0b4
@Sawchord
Move logger up a layer
6dbd40e
@Sawchord
WIP refactoring asset upload into multiple smaller methods
52b9415
@Sawchord
WIP factoring out asset functionality into it's own methods
9926c6b
@Sawchord
WIP refactoring asset upload
2c9629d
@Sawchord
Remove unused mut
8eb7f74
@Sawchord
Implement check evidence method
8a953af
@Sawchord
Implement request_commit_batch method
84ea59e
@Sawchord
WIP improve UX around asset
c1710c7
@Sawchord
WIP improving UX around assets
c2a7737
@Sawchord
WIP UX in assets improvements
0d54940
@Sawchord
WIP implement asset check
ea13a3d
@Sawchord
Test asset check
1df44ee
@Sawchord
Update README
552da41
@Sawchord
Update version and readme
fcd6211
@Sawchord
Update approval UX
929d4af
@Sawchord
Implement a request_prepare_permission method
32a8423
@Sawchord
Functionality to request prepare permission
cbf9848
@Sawchord
WIP Implement lookup for sources
977d4dc
@Sawchord
Test the new functionality
834c07c
@Sawchord
Merge branch 'main' into leon/dfx-orbitv0.2
a75ab04
@Sawchord
Settable verbosity levels
4d5259d
@Sawchord
Merge branch 'main' into leon/dfx-orbitv0.3
25c5e87
@Sawchord
Settable --station
9a3cea2
@Sawchord
WIP improve review UX
301f362
@Sawchord
Some renaming
acb49eb
@Sawchord
WIP implement display for review list
8b1aef0
@Sawchord
Small improvements to the list arg parsing
ff10355
@Sawchord
Improve review list output
f5f0322
@Sawchord
Finish verify get-prepare-permission
6b35472
@Sawchord
Refactor dfx orbit integration tests
b0a46d7
@Sawchord
Refactor dfx-orbit integration tests
d9567bf
@Sawchord
WIP improve asset tests
e143d1b
@Sawchord
WIP making asset integration tests more E2E
bf9a4f5
@Sawchord
Refactoring asset storage
377f8be
@Sawchord
Implement asset verification test
16c1ce1
@Sawchord
Bump dfx-orbit version number
615807f
@Sawchord
Small refactor to improve legibility in dfx-orbit integration tests
9e0409f
@Sawchord
WIP revoking prepare permission
b9215f2
@Sawchord
WIP implement asset better asset permissions
aea5d90
@Sawchord
Better implementation of requesting and verifying asset permissions
257a81c
@Sawchord
WIP implement display for verify commands
ad76aa8
@Sawchord
Implement more display args functions
909ffd1
@Sawchord
Remove the displays again (not sustainable)
79c9b9b
@Sawchord
WIP implement asset commit
4122779
@Sawchord
Implement commit
4ccf3c9
@Sawchord
Implement delete_batch_input
ea0efa0
@Sawchord
Change verify CLI to be easier to modify later on
07dcc59
@Sawchord
Implement cancel commit
733d67c
@Sawchord
Set all batch_ids via flag
b17a926
@Sawchord
Update Readme
9440ca9
@Sawchord
Merge repro fix
a8808ad
@Sawchord
Merge branch 'main' into leon/dfx-orbitv0.5
4d6d294
@Sawchord
Clippy fix
2b3270d
@Sawchord Sawchord marked this pull request as ready for review October 2, 2024 19:07
@Sawchord Sawchord requested review from mraszyk and a team as code owners October 2, 2024 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mraszyk mraszyk Awaiting requested review from mraszyk mraszyk is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Sawchord