remove code that keeps the private key (account) data in memory #40

Closed
trentlarson wants to merge 1 commits from no-accounts-in-memory into master
Owner

Note that this change resulted in errors... for some reason it couldn't find those new methods until I prefixed with "this." (in this PR).

Note that [this change](https://gitea.anomalistdesign.com/trent_larson/kick-starter-for-time-pwa/blame/branch/master/src/views/HomeView.vue#L319) resulted in errors... for some reason it couldn't find those new methods until I prefixed with "this." (in this PR).
trentlarson added 1 commit 1 year ago
trentlarson requested review from anomalist 1 year ago
trentlarson added 1 commit 1 year ago
anomalist added 1 commit 1 year ago
Owner

Understood. Interesting changes. @trentlarson

Understood. Interesting changes. @trentlarson
anomalist added 1 commit 1 year ago
trentlarson added 1 commit 1 year ago
trentlarson added 1 commit 1 year ago
trentlarson added 1 commit 1 year ago
trentlarson changed title from WIP: remove code that keeps the private key (account) data in memory to remove code that keeps the private key (account) data in memory 1 year ago
trentlarson added 2 commits 1 year ago
Poster
Owner

This is ready.

@anomalist Note the problem described above with new methods, fixed in this commit: 624abbb179

This is ready. @anomalist Note the problem described above with new methods, fixed in this commit: https://gitea.anomalistdesign.com/trent_larson/kick-starter-for-time-pwa/commit/624abbb179427eafb3ba65801e7009105b9f5ec2
anomalist added 1 commit 1 year ago
anomalist added 1 commit 1 year ago
anomalist added 1 commit 1 year ago
Owner

@trent_larson/Owners I'm going to close this since I hand merged it.

@trent_larson/Owners I'm going to close this since I hand merged it.
anomalist closed this pull request 1 year ago

Reviewers

anomalist was requested for review 1 year ago
Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.