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

🚀 [QA] Update release environment #703

Merged
merged 39 commits into from
Nov 16, 2023
Merged

🚀 [QA] Update release environment #703

merged 39 commits into from
Nov 16, 2023

Conversation

github-actions[bot]
Copy link

This is a pull request that upon merging will update production environment with recent stage-live changes.
The environment that will be updated:

Read more: Deployment to Production Flow

Karolina Kosiorowska and others added 30 commits November 2, 2023 10:29
Make input component handle value changes with bigints not strings.
Adjust components using the `TokenAmountInput` to new format.
`bigIntToDisplayUserAmount` should be able to display nice label for values
smaller than desired precision, let's make it to show the smallest
possible number like: `<0.01` for desired precision `2` etc.
Allow using "Max" button and don't allow the input to
override the real value with stringified value with less precision.
jagodarybacka and others added 6 commits November 15, 2023 18:53
Added "step=any" to hide the tooltip on the "number" input
Closes #607

#### What else should be done
- [x] Setting the correct animation for bubbles.
- [x] If possible, add animations for `RealmsBar`. `RealmBarIcon` should
move more smoothly when the population is changed.
- [ ] After joining the realm, show a bubble instantly.
- [ ] Improve intervals for realms.
- [x] Adding bubble to the modal window.


### What

The app should be more lively. This PR adds +1 bubble over a realm every
time somebody joins. The bubble should be visible over the realm on the
map but also in the modal window. The population is fetched from the
contracts every minute. We shouldn't show 20 bubbles at the same time
and then not show anything for 1 minute. We should spread out the number
of bubbles we show.

- So let's add a variable `displayedPopulation` to increment it and show
the bubbles with the correct interval.
- Intervals should be different for the realms to make the app more
lively.
- When we init game data, we should refresh the `displayedPopulation`
and align it with the true number.

### Testing

Let's use a patch file to increase the population and do tests. Use the
following file:

[increase-population.patch](https://github.com/tahowallet/dapp/files/13249010/increase-population.patch)
The population is increased randomly every minute.

```
git apply increase-population.patch
```

- [ ] When a user joins the region, a bubble should show up immediately
in the modal.
- [ ] Bubbles should occur at different intervals between realms.
- [ ] After refreshing the page, the population should show the correct
number.
Resolves #694

### Problem

We have been using floating point numbers to transform bigints to
strings with 18 decimal places. But it was easy to mess up the precision
because bigints were larger than max safe int. We need a way to be able
to work in the inputs with both very large and very small numbers.

### What has been done
Make input component handle value changes with bigints not strings.
Adjust components using the `TokenAmountInput` to new format.

- show nicely small balances (`<0,0001 TAHO`) -
`bigIntToDisplayUserAmount` should be able to display nice label for
values smaller than desired precision, let's make it to show the
smallest possible number like: `<0.01` for desired precision `2` etc.
- Allow using "Max" button and don't allow the input to override the
real value with stringified value with less precision.
- Add util to display bigints as strings with max precision


![image](https://github.com/tahowallet/dapp/assets/20949277/d9dc904b-e726-4914-b6af-c0a23eef5f31)


### Testing

To test it please use stake and unstake forms. Type some value or click
"max" and try to unstake/stake - check the value that is displayed on
the wallet token allowance screen (make sure you don't have any
allowance set on the TAHO or veTAHO on that account so you'll be able to
test it that way)

- [x] test it with super small amounts (up to 18 decimal places)
- [x] test it with big amounts (for example >=3301 TAHO)
- [x] test it with big amounts with some values on decimal places
(`3000.0000000001`)
This is a pull request that upon merging will update stage environments
with recent `main` changes.
The environments that will be updated:
* Stage live: https://stage-live--taho-development.netlify.app/
* Stage fork: https://stage-fork--taho-development.netlify.app/

Read more: [Deployment to Production
Flow](https://github.com/tahowallet/dapp/blob/main/docs/testing-env.md)
Copy link

netlify bot commented Nov 16, 2023

Deploy Preview for taho-development ready!

Name Link
🔨 Latest commit 1b33c76
🔍 Latest deploy log https://app.netlify.com/sites/taho-development/deploys/655648551e28540008e0fb45
😎 Deploy Preview https://deploy-preview-703--taho-development.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

xpaczka and others added 3 commits November 16, 2023 17:36
Add XP data for week 3 distributions
This is a pull request that upon merging will update stage environments
with recent `main` changes.
The environments that will be updated:
* Stage live: https://stage-live--taho-development.netlify.app/
* Stage fork: https://stage-fork--taho-development.netlify.app/

Read more: [Deployment to Production
Flow](https://github.com/tahowallet/dapp/blob/main/docs/testing-env.md)
@andreachapman
Copy link
Contributor

Release checklist

This release checklist should be performed before release is published.

  • if something is not working please create an issue and link it here. Checkbox
    should be checked only if everything was fine
  • in case of serious issues or doubts you should ask the team first

Approve the PR when the checklist is finished ✅

Merge the PR when it is approved and we want to update the production
environment 🚀

Manual tests

  1. Connect to wallet that previously staked

    • you see the Portal is Open message
    • you can see the map after entering
    • you see your address connected
    • you see the name of your realm
    • you can navigate to other realms but not stake in them
    • you can stake more TAHO in that realm if you're not fully staked
    • you can unstake
  2. Connect to wallet that hasn't staked but has $TAHO

    • you see the Portal is Open message
    • you can see the map after entering
    • you see your address connected
    • you don't see a realm name beside that
    • you can explore realms and stake into one
  3. Connect to wallet that has NO $TAHO (& no $VETAHO)

    • you see the portal is closed message and the wait list button
    • wait list redirect works as expected
    • you can't see the map
  4. Switch connected wallet in the extension

    • dapp reloads and shows the portal screen
    • reloads with Connect Wallet button if you go to a wallet that's not
      connected
    • reloads with that address connected if that's a connected website for
      the address: if it's an eligible address, you see Access Granted &
      Enter the portal
    • reloads with that address connected if that's a connected website for
      the address: if it's an inelligible address, you see the portal is
      closed message and wait list button
  5. Helper Tool

    • doesn't pop up automatically for users who have visited before and
      closed it
    • doesn't pop up automatically for staked users
    • does come up with the correct message when you click it (standard
      "hang in there, Nomad...")
    • does come up properly for a user connecting for the first time and
      running through onboarding
  6. Population

    • shows on each realm
    • shows on the bottom bar
    • is updated appropriately if a new account stakes
  7. Quests/Questline

    • shows on each realm as expected
  8. Disconnect from dapp

    • click disconnect on address drop down and it disconnects and shows the
      portal screen
  9. Connect Wallet

    • with Taho installed and default wallet
    • with Taho installed and not default
    • with Taho not installed and no other - should be directed to the Chrome store to download Taho & given a message in the dapp to do so
    • with Taho not installed and MM installed - should be directed to the Chrome store to download Taho & given a message in the dapp to do so
  10. XP

  • an account with XP to claim sees that under Claimable Rewards on the Realm modal (in pre-prod we can see this on stage-live but might be easiest to only test this when we're testing XP drops)
  • account can claim XP and sees the wallet balance of VETAHO change accordingly (requires that the account earned XP and hasn't claimed it yet so this could be hard to test on each release but when we're testing XP drops, we should do this)
  • leaderboard shows the top 10 earners in the realm - the connected account is shown at the top with their place on the board shown

Copy link
Contributor

@andreachapman andreachapman left a comment

Choose a reason for hiding this comment

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

Approved for prod

@andreachapman andreachapman merged commit e8467fa into release Nov 16, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants