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

Follow-up for https://github.com/haveno-dex/haveno/pull/1030 #1065

Closed
wants to merge 1 commit into from

Conversation

wp07e
Copy link
Contributor

@wp07e wp07e commented Jun 23, 2024

This is an initial "follow-up" for #1030.

I, very "sillily", thought this was a one-line code change but it turns out I was very foolish.

Anyhow, I ran many different kinds of test cases on this one for about 10 mins. That included:

  1. Creating payment accounts.
  2. Creating multiple payment accounts.
  3. Making sure payments persisted selected currencies.
  4. Making sure after "SELL" or "BUY" buttons are pressed, UI reflects selected currencies particularly for the drop-down box. NOTE: There is one small bug where sometimes during the same session or after a restart, the selected currency for a multi-currency account when trying to create a "SELL" or "BUY" offer will be blank. In that case, all you need to do is select you desired currency and that change will persist until you add a new payment account or reboot (in some cases). But this is not a functional issue and does not effect the application in any meaningful way (although it would be nice to fix it)
  5. Making sure everything is still configured correct after a restart.
  6. Restarting multiple times with different payment accounts (new added / some deleted).
  7. Deleting an existing payment account then recreating the same payment account but with different currencies.
  8. Running through the above cases randomly.

The only thing I have not tested is whether this works on the stage net. From what I can tell, although some values are persisted it is only on the user side and does not effect any other users on the network.

Keep in mind that these changes only affect most of the Traditional Currency accounts. It does not effect Crypto Currency account no other non-standard accounts Vecho "HISTSIZE=500" >> ~/.bashrc

@wp07e wp07e closed this Jun 23, 2024
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.

1 participant