-
Notifications
You must be signed in to change notification settings - Fork 102
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
testing: Start app with seed. #1398
Conversation
Something must be missing when restoring an account. Looking into it... |
I believe we need to try and discover the account first. Adding that ability to the rpc server. |
036d55a
to
3a15bd4
Compare
Did not add the exchange config to result because that can be fetched separately with |
3a15bd4
to
8582309
Compare
For completeness, this makes sense to add. For the testing harness, I don't believe |
core's |
Not expected. You should be able to use
|
It still seems to work as I described.
|
and can you login after that? |
Well... after doing that it didn't show the DEX on the page. Then I restarted and I can login but it wants me to setup a DEX server. After I hard refreshed the page it showed though.
|
Hmm, yeah I think it changes state and the frontend doesn't know to do a fetchUser to get a completely updated state. I had to hard refresh to get it to recognize things |
Well, I guess the rpc endpoint wasn't necessary, although it does let you login right after. |
It's good to have for sure. I'm curious why we cannot login right after with Register. We should get to the bottom of that. If we do the dexcctl steps I described prior to opening the browser page, it should let you login. A couple things appear off. If you don't restart there are no exchanges in the /api/user response. When you do restart it keeps going to /register even though if you go to /markets it's all ready at the exchange registered via |
@JoeGruffins pls try out #1407 when you are able |
8582309
to
a97a096
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.
Looks good. Would you please squash the first two rpcserver commits? I'll merge this PR via rebase with two separate commits, one for the rpcserver and one for the harness.
Allows accounts that were previously paid for to be restored. This should be used as apposed to Register when restoring from seed.
a97a096
to
f320920
Compare
Merged the first two. |
closes #1393