-
Notifications
You must be signed in to change notification settings - Fork 192
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
feat: add export account #1012
feat: add export account #1012
Conversation
@@ -195,6 +236,70 @@ function AccountsScreen() { | |||
</div> | |||
</form> | |||
</Modal> | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dylancom you think by now we should move out these two modals into separate component-files?
51f34b4
to
a40a024
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yay!
65282a3
to
c093d7c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got one last thing. and if you fix the merge conflict we are good to go!
61d8ed7
to
f6ebb49
Compare
Ready! |
Please fix conflicts. You think we should move the modals in to a separate file(s)? |
f6ebb49
to
22257bf
Compare
The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should add i18n (@im-adithya maybe we do this in a new PR?)
Sure! |
Describe the changes you have made in this PR
Introduces Export option in the accounts list.
Link this PR to an issue
Closes #669
Type of change (Remove other not matching type)
feat
: New feature (non-breaking change which adds functionality)Screenshots of the changes (If any)
If we have lightning address:
If we don't:
Add screenshots to help explain your problem
How has this been tested?
Ran all tests
Checklist