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

Publish Stage #3927

Merged
merged 8 commits into from
Nov 4, 2024
Merged

Publish Stage #3927

merged 8 commits into from
Nov 4, 2024

Conversation

jonator
Copy link
Member

@jonator jonator commented Nov 1, 2024 •
edited
Loading

Portfolio improvements

  • Mobile support (fix table padding, hide chart, fix CTA buttons size below balance)
  • Fix animating chart when navigating to page and chart is collapsed
  • Add current time when hovering over smaller time spans in chart
  • Fix BG flicker issue
  • Fix flicker with trade page title, remove portfolio page title
  • Dismissible cypher promo card

yury-dubinin and others added 6 commits October 29, 2024 10:42
@yury-dubinin
Increase order amounts (#3921)
490c502
@jonator
Fix: check for 0 before division (#3914)
15f2180 
* check for 0 in tests
@yury-dubinin
Updated Prod E2E tests workflow (#3923)
780e00a
@jonator
Portfolio improvements (#3924)
414bf41 
* Closes FE-1188

* Closes FE-1009

* lowercase open orders

* hide nav bar title

* optimize rendering

* SSR swap tool

* clean up limit order ff & bg

* remove old swap tool code

* add knip JSON

* fix translations test

* coderabbit review
@jonator
Knip (#3925)
5277eab 
* Closes FE-1188

* Closes FE-1009

* lowercase open orders

* hide nav bar title

* optimize rendering

* SSR swap tool

* clean up limit order ff & bg

* remove old swap tool code

* add knip JSON

* knip & lint

* fix translations test

* fix packages

* fix

* clean up localizations
@jonator
Fix layout shift when nav to Portfolio page w collapsed chart (#3926)
b8012ce 
* silence remote patterns warning

* invalidate portfolio query on tx

* mobile, prevent animation, format date on chart hover

* improve mobile table

* less padding around table

* adjust
@vercel Vercel
Copy link

vercel bot commented Nov 1, 2024 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
osmosis-frontend ✅ Ready (Inspect) Visit Preview ✅ 3 resolved Nov 1, 2024 2:35pm
osmosis-frontend-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 1, 2024 2:35pm
osmosis-testnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 1, 2024 2:35pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
osmosis-frontend-datadog ⬜️ Ignored (Inspect) Visit Preview Nov 1, 2024 2:35pm
osmosis-frontend-edgenet ⬜️ Ignored (Inspect) Nov 1, 2024 2:35pm

@jonator jonator changed the title Publish STage Publish Stage Nov 1, 2024
MaxMillington
MaxMillington approved these changes Nov 1, 2024
View reviewed changes
Copy link
Collaborator

@MaxMillington MaxMillington left a comment

Choose a reason for hiding this comment

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

@yury-dubinin having some issues with the e2e tests it seems

@jonator jonator merged commit 5120911 into master Nov 4, 2024
3689 of 3712 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CryptoAssassin1 CryptoAssassin1 CryptoAssassin1 approved these changes

@yury-dubinin yury-dubinin yury-dubinin approved these changes

@JoseRFelix JoseRFelix JoseRFelix approved these changes

@MaxMillington MaxMillington MaxMillington approved these changes

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

Successfully merging this pull request may close these issues.
6 participants
@jonator @MaxMillington @JoseRFelix @yury-dubinin @CryptoAssassin1 @greg-nagy