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 #549

Merged
merged 60 commits into from
Oct 26, 2023
Merged

🚀 [QA] Update release environment #549

merged 60 commits into from
Oct 26, 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

jagodarybacka and others added 30 commits October 24, 2023 18:42
Use locally saved data to get most of the informations about XP
The current tracking is just that the user hit the page. It's double-
tracking on dev, need to check that this doesn't happen in prod.
It feels a little wobbly here, but assuming it works, it should give us
a way to see if people start staking and fail to complete, or same with
unstaking, in case there are wallet issues or similar.
The `github.head_ref` variable does not include the `ref/heads/` part.
The `github.head_ref` variable does not include the `ref/heads/` part.
Ref #503 

Adjusting xp allocations since the start date do not match with
contracts
Ref #503 

What has been done:
- Correct start date was hard-coded
- Calculation of the current season week was adjusted depending on if
the season has already started
Resolves #509 

Adding extra check to not render the "first visit to realm" assistant
pop-up if user is already staked
xpaczka and others added 24 commits October 26, 2023 09:58
Resolves #531 

Change favicon to Subscape
Resolves #520 
Resolves #522 

What has been done:
- Rulebook opens in new tab now
- Changed color of Gitcoin realm
Resolves: #527 

<img width="700" alt="Screenshot 2023-10-26 at 10 14 19"
src="https://github.com/tahowallet/dapp/assets/28560653/8b7406c6-6647-46a2-8d20-da2859d9a90f">
Resolves #516
Resolves #515
Resolves #521 

What has bee done:
- Changed the entering animation of realm modal
Closes #503

For now let's just hardcode the weekly reward.
We never put this explicitly on the roadmap, but it would be good to get
a little better feel for how people are using the dApp, and be able to
compare against the wallet info.

A few notes: 

- I haven't been able to test extensively, and in particular am not
fully convinced of the approach for staking/unstaking. It's built to
track the beginning and end of that process, so we can see if people are
starting to stake/unstake and then not making it through to transaction
completion.
- Priority-wise, I'd like to have pageviews (the first commit) at
launch, then if possible realm opens, and then if we're feeling good
about it the staking/unstaking stuff. To some extent we can do
staking/unstaking on-chain, so it's not quite as critical.
- We'll want to set up the ANALYTICS_ENV and POSTHOG_API_KEY variables
appropriately. Right now I'm just using the API key from taho.xyz. The
default analytics env is DEV, we'll want PROD for the released dApp.

Torch-passing here, let's consider pageview + realm opens as second
priority to any blockers, but higher priority than any nice-to-haves for
launch.

Leaving this in draft as I realize we may need to cherry-pick partial
commits and such to get the outcome we're most comfortable with.
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)
@netlify
Copy link

netlify bot commented Oct 26, 2023

Deploy Preview for taho-development ready!

Name Link
🔨 Latest commit 8c7898c
🔍 Latest deploy log https://app.netlify.com/sites/taho-development/deploys/653a3f1544c1750008991cc7
😎 Deploy Preview https://deploy-preview-549--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.

@michalinacienciala
Copy link
Contributor

michalinacienciala commented Oct 26, 2023

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 with $TAHO

    • you see the Portal is Open message
    • you can see the map after entering
    • you see your address connected
  2. Connect to wallet that previously staked

    • 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
  3. Connect to wallet that hasn't staked but has $TAHO

    • you see your address connected
    • you don't see a realm name beside that
    • you can explore realms and stake into one
  4. 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
  5. 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
  6. Helper Tool (note: this might be hard to test since with new releases, the
    cache will always be fresh and these results may not be true)

    • 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
      "hope you're enjoying...")
    • does come up properly for a user connecting for the first time and
      running through onboarding
  7. Population

    • shows on each realm (how we test that it's accurate, not sure but it
      should show)
    • shows on the bottom bar
  8. Quests/Questline

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

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

  • with Taho installed and default wallet
  • with Taho installed and not default
  • with Taho not installed and no other
  • with Taho not installed and MM installed
  1. Staking
  • stake using an account which hasn't staked before
    • Congratulations page opened after staking
    • Congrats Citizen show by the Assistant
    • Population updated, cooldown period starts from 60 min, staked amount updated
    • Can stake again to the same realm
    • Cannot stake to other realm, Take me to my realm banner shown in Stake tab
  • stake to the same Realm when cooldown period is still ON
    • Congratulations page not opened again
    • Congrats Citizen not show again by the Assistant
    • Population updated, cooldown period updated to show again 60 min, staked amount updated
    • Can stake again to the same realm
    • Cannot stake to other realm, Take me to my realm banner shown in Stake tab
  • stake to other Realm when cooldown period is ended

@michalinacienciala michalinacienciala merged commit 2a2b097 into release Oct 26, 2023
6 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.

5 participants