Skip to content
This repository has been archived by the owner on Jun 16, 2022. It is now read-only.

LIVE-1766 Add ETH goerli #2349

Merged
3 commits merged into from
Apr 29, 2022
Merged

LIVE-1766 Add ETH goerli #2349

3 commits merged into from
Apr 29, 2022

Conversation

hzheng-ledger
Copy link
Contributor

@hzheng-ledger hzheng-ledger requested a review from a team as a code owner March 29, 2022 13:43
@github-actions
Copy link

github-actions bot commented Mar 29, 2022

Thanks for your contribution.
To be groomed for next release, you need to:

  • pass the CI
  • if needed, run /generate-screenshots
  • have a dev review
  • have a QA review

Why /generate-screenshots ?

If your PR contains UI related changes,
it might be necessary to regenerate screenshots.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

@hzheng-ledger
Same as on Desktop I think 🙂

  • Need to re-generate the cryptoassets.md file
  • Maybe need to update the tests/specs/settings/developermode.spec.js test

@hzheng-ledger hzheng-ledger requested a review from a team as a code owner March 31, 2022 09:53
@hzheng-ledger
Copy link
Contributor Author

hzheng-ledger commented Mar 31, 2022

@hzheng-ledger Same as on Desktop I think 🙂

  • Need to re-generate the cryptoassets.md file
  • Maybe need to update the tests/specs/settings/developermode.spec.js test

@haammar-ledger There are no cryptoassets.md and developermode.spec.js test in LLM as far as I am concerned. I think we don't have to update them. Right?

@hzheng-ledger hzheng-ledger requested a review from a user March 31, 2022 09:56
@ghost
Copy link

ghost commented Apr 1, 2022

@hzheng-ledger it seems it was deleted recently 😮 idk why but no need anymore indeed. 🙂

@ghost ghost added the feature-ready label Apr 29, 2022
@ghost ghost changed the title Add ETH goerli LIVE-1766 Add ETH goerli Apr 29, 2022
@ghost ghost merged commit 7cdc06a into develop Apr 29, 2022
@ghost ghost deleted the feat/LL-1766 branch April 29, 2022 11:08
@valpinkman valpinkman mentioned this pull request May 5, 2022
@lambertkevin lambertkevin mentioned this pull request May 12, 2022
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants