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

fix degen <-> wdegen #6087

Merged
merged 23 commits into from
Sep 13, 2024
Merged

Conversation

greg-schrammel
Copy link
Contributor

@greg-schrammel greg-schrammel commented Sep 6, 2024

Fixes APP-####

What changed (plus any additional context for devs)

adds degen native asset address

Screen recordings / screenshots

Screen.Recording.2024-09-10.at.12.53.07.mov

What to test

Copy link

linear bot commented Sep 6, 2024

@brunobar79
Copy link
Member

Launch in simulator or device for ed0631a

@brunobar79
Copy link
Member

Launch in simulator or device for e7e42fa

@brunobar79
Copy link
Member

Launch in simulator or device for 5456525

ibrahimtaveras00 and others added 18 commits September 11, 2024 19:12
* fix

* test

* revert

* @walmat related fixes

* @walmat related fixes

* okay disabling the test again, the experimental flag didnt fix it

* okay disabling the test again, the experimental flag didnt fix it

* fix disabled NFTs in testing and rely on remote config for prod

* .
* change to id

* also fix clicks
* add rc-push script

* source .env
…6072)

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 3 to 4.1.7.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v3...v4.1.7)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* change opacity on mwp route source too

* switch to includes
* fix missing dapp metadata for eth actions

* use isHandshakeAction helper function

* Update src/components/MobileWalletProtocolListener.tsx

Co-authored-by: Bruno Barbieri <1247834+brunobar79@users.noreply.github.com>

---------

Co-authored-by: Bruno Barbieri <1247834+brunobar79@users.noreply.github.com>
…6072)

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 3 to 4.1.7.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v3...v4.1.7)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…6072)

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 3 to 4.1.7.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v3...v4.1.7)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* fix

* lint
* done

* change userAssetsStoreCache type

* mid refactor

* refactor

* dont cast to Address

* more

* more

* rm address param

* fix caching

* select address

* comments

* hardhat

* lint
* add logging for pkey errors

* fix keychain error handling

* checksum address before keychain lookup

* remove invalid chainId param from tx

* Update src/keychain/index.ts

* Update src/keychain/index.ts

Co-authored-by: Jin <jin.chung17@gmail.com>

* Explicitly return UserCanceled error for code strings 10 and 13

* Remove returning -3 explicit error code

* Use enum values instead of primitives for error codes for readability

---------

Co-authored-by: Jin <jin.chung17@gmail.com>
@brunobar79
Copy link
Member

Launch in simulator or device for 6e8752f

@brunobar79
Copy link
Member

Launch in simulator or device for fdfe974

@walmat walmat merged commit 64d1387 into develop Sep 13, 2024
8 checks passed
@walmat walmat deleted the gregs/app-1839-no-route-found-wdegen-to-degen branch September 13, 2024 13:59
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.

8 participants