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

Code refactoring as follow up to recent BlockchainType changes #610

Merged
merged 7 commits into from
Jul 18, 2024

Conversation

Oleg-Pecheneg
Copy link
Contributor

No description provided.

@Oleg-Pecheneg Oleg-Pecheneg requested a review from rommex July 17, 2024 12:58
bio == nil &&
location == nil &&
website == nil
guard avatarData == nil else { return false }
Copy link
Collaborator

Choose a reason for hiding this comment

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

:)

return (token == CryptoSender.SupportedToken.eth && chainSpec.chain.identifyBlockchainType() == .Ethereum) ||
(token == CryptoSender.SupportedToken.matic && chainSpec.chain.identifyBlockchainType() == .Matic) ||
(token == CryptoSender.SupportedToken.eth && chainSpec.chain.identifyBlockchainType() == .Base)
switch (token, chainSpec.chain.identifyBlockchainType()) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Beautiful 👏🏼

@Oleg-Pecheneg Oleg-Pecheneg merged commit e757c3c into development Jul 18, 2024
@Oleg-Pecheneg Oleg-Pecheneg deleted the dev/feat/network-type-refactoring branch July 18, 2024 06:30
rommex added a commit that referenced this pull request Jul 26, 2024
* development: (35 commits)
  bumped version to 5.3.5(3)
  MOB-2119 - Maintenance mode (#616)
  MOB-2129 - Fixed occasional camera crash (#615)
  MOB-2111 - Fixed UI and copies in activity filters (#614)
  MOB-2126 - Support ethTransaction for ULW (#613)
  MOB-2111 - Activities filter (#612)
  MOB-2107 Fix Ledger Live to be discoverable (#611)
  Code refactoring as follow up to recent BlockchainType changes (#610)
  bumped version to 5.3.5 (2)
  MOB-2097 Blockchains refactoring (#608)
  Switch to Profiles API for domain name resolution (#609)
  MOB-2104 - Fixed messaging nav title visibility (#606)
  MOB-2104 - Fixed messaging nav bar when search is active (#605)
  MOB-2058 - Revise copy and move Login CTA (#604)
  MOB-2106 - Removed Spot wallet from the list of supported (#603)
  MOB-2104 - Enable Apps Inbox for ULW (#602)
  MOB-2103 - Do not auto-enable groups messaging profile for ULW (#601)
  MOB-2060 Send crypto on Base chain (#579)
  MOB-2102 - Handle unsupported WC requests for ULW (#600)
  Fixed tests target (#599)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants