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

add 24 word mnemonic phrase to recover-account #1070

Merged
merged 17 commits into from
Jan 16, 2024

Conversation

piyalbasu
Copy link
Contributor

Screen.Recording.2023-12-22.at.6.11.31.PM.mov

Copy link

socket-security bot commented Dec 22, 2023

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@stellar/design-system 1.0.1...1.1.2 None +0/-2 2.98 MB stellar-npm-ci

@@ -6,7 +6,7 @@
position: fixed;
top: 0;
width: 100%;
transition: opacity var(--anim-transition-default);
transition: opacity var(--dropdown-animation);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

apparently this var doesn't exist in the repo anymore, so replacing it with this other animation timing var

Base automatically changed from release/5.12.1 to master January 2, 2024 22:27
Copy link

socket-security bot commented Jan 2, 2024

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

@piyalbasu piyalbasu changed the base branch from master to release/5.13.0 January 11, 2024 22:02
...options,
// Use packageFilter to process parsed `package.json` before the resolution (see https://www.npmjs.com/package/resolve#resolveid-opts-cb)
packageFilter: (pkg) => {
if (pkg.name === "@stellar/design-system") {
Copy link
Contributor Author

@piyalbasu piyalbasu Jan 16, 2024

Choose a reason for hiding this comment

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

interesting use-case here:

Webpack/Rollup default to using the file listed under module in package.json, but Jest uses main. I set this up because I'd like Jest to use the same file path that Webpack will be using

@@ -12,9 +12,6 @@
"extension",
"@stellar/freighter-api"
],
"resolutions": {
"**/terser-webpack-plugin": "^1.4.3"
Copy link
Contributor Author

@piyalbasu piyalbasu Jan 16, 2024

Choose a reason for hiding this comment

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

Webpack 5 comes with a newer version of this included. This was causing an issue as this old version of Terser contained an outdated JS spec

await page.locator(".RecoverAccount__phrase-toggle > label").click();

// time out to accomodate animation
await page.waitForTimeout(150);
Copy link
Contributor

Choose a reason for hiding this comment

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

this probably works well enough but wanted to point out that there is also page.waitForSelector('yourselector') and page.waitForLoadState('domcontentloaded') which could give you less of a racy condition to wait for in the case of waiting for css animations.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, nice! That's much better. I was not psyched about doing this, but couldn't get it to work reliably otherwise

@piyalbasu piyalbasu merged commit 1e924b3 into release/5.13.0 Jan 16, 2024
3 checks passed
@piyalbasu piyalbasu deleted the feature/24-word-mnemonic branch January 16, 2024 21:49
piyalbasu added a commit that referenced this pull request Jan 18, 2024
* Bump follow-redirects from 1.15.2 to 1.15.4 (#1080)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.2 to 1.15.4.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.2...v1.15.4)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

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

* Bump follow-redirects from 1.15.2 to 1.15.4 in /docs (#1079)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.2 to 1.15.4.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.2...v1.15.4)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

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

* add 24 word mnemonic phrase to recover-account (#1070)

* add 24 word mnemonic phrase to recover-account

* resolve to `module` entrypoint in SDS

* Bump versions to 5.12.1 (#1076)

* docs(): bumping release to 5.12.1

* Empty-Commit

---------

Co-authored-by: GitHub Action <action@github.com>
Co-authored-by: Piyal Basu <pbasu235@gmail.com>

* combine steps in order to dedupe build steps

* adds playwright browser install step

* undo early merge

* undo jest upgrade

* revert yarn

* yarn.lock for sds

* fix out of date terser plugin

* add test

* add comment

* rm race condition in test

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: GitHub Action <action@github.com>
Co-authored-by: aristides <aristides.staffieri@stellar.org>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: GitHub Action <action@github.com>
Co-authored-by: aristides <aristides.staffieri@stellar.org>
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