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.

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

chore: polish multichain Transak upgrade #2788

Conversation

jeeanribeiro
Copy link
Member

Summary

closes #2785

Testing

Platforms

  • Desktop
    • MacOS
    • Linux
    • Windows

Checklist

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or modified tests that prove my changes work as intended
  • I have verified that new and existing unit tests pass locally with my changes
  • I have verified that my latest changes pass CI workflows for testing and linting
  • I have made corresponding changes to the documentation

@jeeanribeiro jeeanribeiro self-assigned this Aug 9, 2024
@nicole-obrien nicole-obrien marked this pull request as ready for review August 13, 2024 14:44
@nicole-obrien nicole-obrien merged commit 08bcd28 into 2590----create-buysell-exchange-ui Aug 13, 2024
10 checks passed
@nicole-obrien nicole-obrien deleted the 2785-polish-multichain-transak-upgrade branch August 13, 2024 14:47
jeeanribeiro added a commit that referenced this pull request Aug 14, 2024
* feat: adds MVP exchange screen in buy/sell view

* feat: adds multichain support in Buy/Sell view (#2725)

* feat: adds multichain support in BuySell view

* refactor: use network id on transak cryptocurrency

Co-authored-by: Jean Ribeiro <[email protected]>

* use id in token popup filter

* enable buy sell on all profiles

* add network into transak crypto currencies

* dynamically pull in transak cryptos based of profile networks

* enhancment: sort transak cryptos

---------

Co-authored-by: Nicole O'Brien <[email protected]>

* fix: ui polish

* fix: select token popup changing store

* feat: adds icon and skeleton loading in cryptocurrency tile

* feat: adds getPrice function

* feat: add quotes panel in buy/sell view (#2739)

* WIP

* chore: fix reactivity and organize code

* chore: fix selected cryptocurrency & increase exchange panel height

* feat: add RecipientInput

---------

Co-authored-by: Nicole O'Brien <[email protected]>

* feat: Transak window as a Popup (#2743)

* feat: Transak window as a Popup

* refactor: change TransakConnectionPanel to be a popup banner

* fix: disable button if necessary

* fix: adds locale strings in quotations panel

* fix: makes TransakFramePopup props optional

* enhancement: sets error to true if any of the props in falsy

* feat: buy/sell quotations timer (#2753)

* feat: adds buy/sell quotations timer

* fix: quote timer logic

Co-authored-by: Jean Ribeiro <[email protected]>

* refactor: cleanup quote loading state

---------

Co-authored-by: Nicole O'Brien <[email protected]>

* feat: adds selectable quotation and dynamic text (#2759)

Co-authored-by: Nicole O'Brien <[email protected]>

* feat: localise payment methods, improve icons and fix iframe width/border (#2760)

* enhancement: adds payment methods localization

* fix: remove array of undefined in quotes

* fix: getComputedStyle error and iframe out of bounds

* chore: update bloom ui kit version

---------

Co-authored-by: Nicole O'Brien <[email protected]>

* enhancement: auto select first quotation

* feat: adds min and max amount in TransakAmountInput (#2761)

* feat: adds min and max amount in TransakAmountInput

* fix: logic so that no quotes are generated when value is outside min or max

Co-authored-by: Jean Ribeiro <[email protected]>

---------

Co-authored-by: Nicole O'Brien <[email protected]>

* chore: polish multichain Transak upgrade (#2788)

* fix: only show currencies that have payment method available

* feat: adds TransakCryptoCurrencyTile variant

* feat: adds view title

* feat: adds colors to tokens network pills

* fix: prevent selected currency option undefined error

* feat: adds EmptyListPlaceholder in TransakExchangePanel

* refactor: uses transak api base url depending on profile network

* fix: undefined currency

* feat: adds transak widget url for testnet and mainnet

* fix: quotations positioning

* feat: updates ui kit and adds Pill colors

* chore: inverts TransakCryptoCurrencyAmountTile amounts position

* refactor: create network name pill component

* ui: fix transak frame height

* refactor: organise transak query params, enable transak menu

* refactor: rename transak quotation tile

* fix: iota showing on buy sell for mainnet profiles

* ui: dont capitalise names on network pill

---------

Co-authored-by: Nicole O'Brien <[email protected]>

---------

Co-authored-by: Nicole O'Brien <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants