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

Make sure accounts is always an Array #26

Merged
merged 3 commits into from
Nov 19, 2018
Merged

Make sure accounts is always an Array #26

merged 3 commits into from
Nov 19, 2018

Conversation

brunobar79
Copy link
Contributor

@brunobar79 brunobar79 commented Nov 15, 2018

I was doing keyring.getAccounts().map but when you're starting from scratch there's no keyring and undefined.map() breaks.

@brunobar79 brunobar79 changed the title make sure accounts is always array Make sure accounts is always an Array Nov 15, 2018
@codecov-io
Copy link

codecov-io commented Nov 15, 2018

Codecov Report

Merging #26 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #26   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          18     18           
  Lines         889    889           
  Branches       97     97           
=====================================
  Hits          889    889

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c1360f3...1867fe3. Read the comment docs.

Copy link
Contributor

@bitpshr bitpshr left a comment

Choose a reason for hiding this comment

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

One stylistic comment, but this shouldn't hold up the PR. Looks good.

src/KeyringController.ts Outdated Show resolved Hide resolved
@brunobar79 brunobar79 merged commit e2098be into master Nov 19, 2018
@whymarrh whymarrh deleted the keyring-fix-1 branch November 5, 2019 17:43
rickycodes added a commit that referenced this pull request Nov 23, 2022
* 32.0.8

* update changelog

Co-authored-by: github-actions <github-actions@github.com>
Co-authored-by: Ricky Miller <ricky.miller@gmail.com>
mcmire added a commit to mcmire/core that referenced this pull request Jul 17, 2023
Published versions of `superstruct` between 0.16.1 and 0.16.4 contained
syntax (`??=`) that Node did not support until v15. This effectively
changed the minimum required Node version for this package, and any
other package using this package, to v15. A [fix for this issue][1]
landed in `superstruct` 0.16.5, so this commit bumps that library
accordingly so as to restore compatibility with Node 14.

[1]: ianstormtaylor/superstruct#1112
kanthesha pushed a commit that referenced this pull request Sep 19, 2023
Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.12.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](ajv-validator/ajv@v6.12.2...v6.12.6)

---
updated-dependencies:
- dependency-name: ajv
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
MajorLift pushed a commit that referenced this pull request Sep 22, 2023
---------

Co-authored-by: github-actions <github-actions@github.com>
Co-authored-by: legobt <6wbvkn0j@anonaddy.me>
Co-authored-by: legobeat <109787230+legobeat@users.noreply.github.com>
Co-authored-by: Maarten Zuidhoorn <maarten@zuidhoorn.com>
kanthesha pushed a commit that referenced this pull request Oct 11, 2023
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
* make sure accounts is always array

* use await instead of yield

* Use Array.isArray instead of array.length
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.12.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](ajv-validator/ajv@v6.12.2...v6.12.6)

---
updated-dependencies:
- dependency-name: ajv
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
* make sure accounts is always array

* use await instead of yield

* Use Array.isArray instead of array.length
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.12.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](ajv-validator/ajv@v6.12.2...v6.12.6)

---
updated-dependencies:
- dependency-name: ajv
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
MajorLift pushed a commit that referenced this pull request Oct 12, 2023
Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.12.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](ajv-validator/ajv@v6.12.2...v6.12.6)

---
updated-dependencies:
- dependency-name: ajv
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

4 participants