-
-
Notifications
You must be signed in to change notification settings - Fork 275
feat(perps): sync PerpsController implementation for npm publishing #7941
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.
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
Open
abretonc7s
wants to merge
28
commits into
main
Choose a base branch
from
feat/perps/controller-migration-sync
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+32,824
−348
Open
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
ecfd291
feat(perps): add missing dependencies and tsconfig references
abretonc7s f2bd59f
feat(perps): mobile sync
abretonc7s 00f41ef
fix(perps): harden toggleTestnet and depositWithConfirmation edge cases
abretonc7s 7667887
fix(perps): mark deposit request as failed on pre-submission errors
abretonc7s 49f7382
fix(perps): cache standalone provider and reorder switchProvider checks
abretonc7s 8478e7c
fix(perps): resolve TS2345 type error in depositWithConfirmation
abretonc7s 93f4f50
fix(perps): align tsconfig.json references with tsconfig.build.json
abretonc7s a5faeb4
fix(perps): add placeholder test to unblock CI
abretonc7s 771943a
fix(perps): register messenger actions and clean up feature-flag subs…
abretonc7s 2d939d4
fix(perps): fix stale state bugs in feature flags, deposit lifecycle,…
abretonc7s 934f6bb
Merge remote-tracking branch 'origin/main' into feat/perps/controller…
abretonc7s af32301
fix(perps): satisfy lint rules in placeholder test
abretonc7s 2ed4613
fix: lockfile
abretonc7s 33a443d
fix(perps): add generated method action types for PerpsController
abretonc7s f683bcd
fix(perps): add changelog entry for method action types
abretonc7s f6f3ba0
feat(perps): remove feature controller coupling via platform dependen…
abretonc7s fd89cb1
feat: latest mobile sync
abretonc7s ef4580c
Merge remote-tracking branch 'origin/main' into feat/perps/controller…
abretonc7s a6424be
fix: linting
abretonc7s 567bc78
Merge remote-tracking branch 'origin/main' into feat/perps/controller…
abretonc7s 6b0c888
refactor(perps): migrate from DI controllers to messenger pattern
abretonc7s 7fc9210
refactor(perps): move messenger types to separate internal file
abretonc7s 09603a5
refactor(perps): use PerpsAddTransactionOptions in #submitTransaction
abretonc7s 7e7a046
Merge branch 'main' into feat/perps/controller-migration-sync
gambinish bc724fa
chore: update yarn.lock
gambinish b195506
fix: constraints
gambinish 875b173
chore: dedupe
gambinish b014892
chore: update type
gambinish File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| { | ||
| "lastSyncedMobileCommit": "9c2fbb121e832c2f7f4012a132b1b87eda9b6758", | ||
| "lastSyncedMobileBranch": "feat/perps/di-refactor-remove-controller-deps", | ||
| "lastSyncedCoreCommit": "7fc921027bd37a4acebf1c48a7c18a394e6e549e", | ||
| "lastSyncedCoreBranch": "feat/perps/controller-migration-sync", | ||
| "lastSyncedDate": "2026-02-26T14:23:21Z", | ||
| "sourceChecksum": "7c15b984a902c2543d8c846b0f720c2aa4f5ed0a35dd03140c0481216ed71b34" | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Should this be gitignored? Or is it needed as source code to make the sync script work