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(pallet-gear-builtin): Add proxy builtin #4259

Merged
merged 15 commits into from
Oct 24, 2024
Merged

feat(pallet-gear-builtin): Add proxy builtin #4259

merged 15 commits into from
Oct 24, 2024

Conversation

techraed
Copy link
Member

@techraed techraed commented Sep 25, 2024 •
edited
Loading

This PR basically adds proxy builtin which allows programs to add delegatee for the actions to be done on their behalf.

TODO:

  • add actual test that proxy on behalf of program works
  • add docs

@techraed techraed added A1-inprogress Issue is in progress or PR draft is not ready to be reviewed D2-node Gear Node labels Sep 25, 2024
@techraed techraed self-assigned this Sep 25, 2024
@techraed techraed changed the title feat(pallet_gear_builtin): Add proxy builtin feat(pallet-gear-builtin): Add proxy builtin Sep 26, 2024
@techraed techraed added A0-pleasereview PR is ready to be reviewed by the team and removed A1-inprogress Issue is in progress or PR draft is not ready to be reviewed labels Sep 26, 2024
@techraed techraed mentioned this pull request Sep 30, 2024
Closed
breathx
breathx approved these changes Oct 1, 2024
View reviewed changes
pallets/gear-builtin/src/lib.rs Outdated Show resolved Hide resolved
@techraed techraed added A2-mergeoncegreen PR is ready to merge after CI passes and removed A0-pleasereview PR is ready to be reviewed by the team labels Oct 11, 2024
@techraed techraed merged commit 73929e3 into master Oct 24, 2024
11 checks passed
@techraed techraed deleted the st-builtin-proxy branch October 24, 2024 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@breathx breathx breathx approved these changes

@gshep gshep Awaiting requested review from gshep

@ekovalev ekovalev Awaiting requested review from ekovalev

@StackOverflowExcept1on StackOverflowExcept1on Awaiting requested review from StackOverflowExcept1on

Assignees

@techraed techraed

Labels
A2-mergeoncegreen PR is ready to merge after CI passes D2-node Gear Node
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@techraed @breathx @StackOverflowExcept1on