-
Notifications
You must be signed in to change notification settings - Fork 24
SOV-5275: remove bob pool migration mentions #1129
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
SOV-5275: remove bob pool migration mentions #1129
Conversation
🦋 Changeset detectedLatest commit: 6bcb974 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
✅ Deploy Preview for sovryn-storybook canceled.
|
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.
Pull request overview
This PR removes references to the BOB pool migration feature, which appears to be no longer needed. The changes involve removing UI components, translation strings, and migration-related functionality.
Key Changes:
- Removed the BOBMigrationBanner component and its usage
- Removed migration-related UI elements from the DeprecatedBadge component
- Cleaned up unused translation keys
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| apps/frontend/src/locales/en/translations.json | Removed the "migrate" translation key that was used for migration UI |
| apps/frontend/src/app/5_pages/MarketMakingPage/components/AmbientMarketMaking/components/BOBMigrationBanner/BOBMigrationBanner.tsx | Deleted the entire BOBMigrationBanner component file |
| apps/frontend/src/app/5_pages/MarketMakingPage/components/AmbientMarketMaking/AmbientMarketMaking.tsx | Removed import and usage of BOBMigrationBanner component |
| apps/frontend/src/app/2_molecules/DeprecatedBadge/DeprecatedBadge.tsx | Removed the "Migrate" button and related imports from the badge |
| .changeset/quiet-mirrors-rule.md | Added changeset documenting this removal |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
apps/frontend/src/app/2_molecules/DeprecatedBadge/DeprecatedBadge.tsx
Outdated
Show resolved
Hide resolved
✅ Deploy Preview for sovryn-dapp ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
https://sovryn.atlassian.net/browse/SOV-5275