-
Notifications
You must be signed in to change notification settings - Fork 4
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 #886
Conversation
When user was typing the balance amount manually then input was not updating values correctly. We moved to `useCallback` instead of `useEffect` and fixed the condition for updating parent component with new values.
We had a mistake with the expected behavior.
…870) We had a mistake with the expected behavior.
Resolves #866 ## What has been done Messages from assistant were not displayed correctly due to unnecessary `useEffect`. Also extracted some `Island` logic to `useOnRealmClick` hook. ## Testing - [x] Clear `localStorage` - [x] Connect wallet that is not staked in any realm - [x] After entering portal, `Welcome to Subscape, Nomad` message appears - [x] After clicking on realm for first time, `Why join a realm` message appears
### What Prepare app for the end of beta season, changes when `IS_BETA_CLOSED = true` are: - countdown shows time left to claim XP ![image](https://github.com/tahowallet/dapp/assets/20949277/2c8051f0-e72f-420d-afe0-9e08de77a6e2) - there are no challanges visible ![image](https://github.com/tahowallet/dapp/assets/20949277/ad05c822-0cd1-4aac-89a8-61b67c825f64) - staking is impossible ![image](https://github.com/tahowallet/dapp/assets/20949277/0e1cae78-19bd-4fa7-a754-365161cb63d1) - There is a hint below the claimable XP ![image](https://github.com/tahowallet/dapp/assets/20949277/f7adfd92-1b95-4c1a-9b4a-3a31f914df6a) - there are no "stake to join realm" or "you are already staked in another realm" banners visible (as it is no longer possible to stake) ### Testing - [x] with `IS_BETA_CLOSED = true` test changes listed above - [x] with `IS_BETA_CLOSED = false` - there should be no changes compared to prod env
Resolves #764 ### What When user was typing the balance amount manually then input was not updating values correctly. We moved to `useCallback` instead of `useEffect` and fixed the condition for updating parent component with new values. The issue was caused mostly by incorrect condition but I wanted to get rid of `useEffect` as the dependencies array was intentionally incomplete and it seems like it wasn't the most correct hook anyway. ### Testing General regression testing is needed here (@michalinacienciala). Let's not merge unless it will be tested for regressions. - [x] make sure scenarios written in the task above are working correctly now - [x] test staking and unstaking both full and partial amounts - [x] make sure we can stake/unstake values with decent precision (either by typing or using "max" button) ![image](https://github.com/tahowallet/dapp/assets/20949277/c4756451-b6eb-425b-8641-fea353ed4f6d)
## Issues connected Ref #875 Ref #876 Resolves #878 ## What has been done - Added logic for sign up form - Added message for sign up form (success, failed, invalid email) ## Testing - [x] Entering no email - "Invalid email" message is displayed - [x] Entering not valid email - "Invalid email" message is displayed - [x] Entering valid email - "Sign up successful" message is displayed - [x] If sign up successful, email input is cleared - [x] All messages (success or fail) disappears after 3 seconds - [x] After sign up, email used will be visible in customer.io -> People ## Screenshots / images / videos ![Zrzut ekranu 2023-12-12 o 15 02 53](https://github.com/tahowallet/dapp/assets/73061939/8e1cd934-c08d-44d4-81c9-372d8bbe2f2d)
## Issues connected Resolves #863 ## What has been done Modified text rendered when connecting dapp to Taho wallet ## Testing - [x] Connect your wallet, the text should be "Granting access to your account." ## Screenshots / images / videos ![Zrzut ekranu 2023-12-13 o 16 32 45](https://github.com/tahowallet/dapp/assets/73061939/2079b564-4b1d-4daf-94ad-0c105e1247f1)
## What has been done Week counter went above season duration, changed to be 7/7 ## Testing - [x] Set `BETA_IS_CLOSED` flag to `true` - current week is 7/7 ## Screenshots / images / videos ![Zrzut ekranu 2023-12-14 o 10 15 54](https://github.com/tahowallet/dapp/assets/73061939/0c2123c4-49ac-4eb2-a9c1-c494fc128e48)
## Issues connected Resolves #823 ## What has been done - Fixed ENS and UNS names not updated automatically (without reload) - Added `useCachedWalletName` hook ## Testing - [x] Clear `localStorage` - [x] Connect wallet - [x] After a short interval, account name is automatically updated - [x] Reload the page, the name is still visible
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)
✅ Deploy Preview for taho-development ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Release checklistThis release checklist should be performed before release is published.
Approve the PR when the checklist is finished ✅ Merge the PR when it is approved and we want to update the production Manual tests*many of these tests can be combined with each other as you're testing. I suggest looking through the whole checklist first then executing tests in a way that allows you to check off multiple items at once. Some suggestions are provided.
|
## What has been done Fixes error with `ens` destructuring ## Screenshots / images / videos ![image](https://github.com/tahowallet/dapp/assets/73061939/52178c41-afa2-4ee8-82be-02d1738e6091)
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)
## What has been done Hidden some of assistant messages ## Screenshots / images / videos ![image](https://github.com/tahowallet/dapp/assets/73061939/d07c0305-8e22-4e1b-852f-90626b9fe151) ![image](https://github.com/tahowallet/dapp/assets/73061939/f4d9a1ed-d3ef-4593-92a1-b3cb84abd2e2)
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)
We are missing Gitcoin - but the rest can be merged to stage already
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)
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)
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