-
Notifications
You must be signed in to change notification settings - Fork 7
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
v230711-capture-app-ionic-launch (postpone from 0627) #2840
Comments
➤ Kenny Hung commented: According to https://dt42-numbers.slack.com/archives/C0323488MEJ/p1687751911165579 ( https://dt42-numbers.slack.com/archives/C0323488MEJ/p1687751911165579 ) push this release to 0703 sprint. |
➤ Kenny Hung commented: Sam (cc Scott YanTammy Yang) QA Warm reminder: If the release time is going to be later than 16:00 TPE, please notify QA in advance. Thank you. |
➤ Sam commented: Kenny Hung, release reminder submitted ( https://dt42-numbers.slack.com/archives/CUSDE44MD/p1689058398412719 ). I also updated iOS minimum requirement to 15.6. Since this release contains ✓ [code-quality] Updating from Capacitor 4 to Capacitor 5 ( https://app.asana.com/0/0/1204798271886976 ) we also need to test other parts like push notification etc. |
➤ Kenny Hung commented: Sam (cc Tammy YangScott Yan) Because this release is postponed from previous sprint, this release should be launch release. |
➤ Sam commented: Kenny Hung do I miss something? |
➤ Kenny Hung commented: Sam No, just sync the information about this release. |
➤ Kenny Hung commented: SamBofu Chen (cc Tammy Yang) I found [code quality] signature/verification update unit tests ( https://app.asana.com/0/1201016280880500/1204317281149648/f ) Does it need to rearrange in the future sprint? |
➤ Sam commented: Kenny Hung it can be finished it this sprint (i didn't add it to firebase release since It does not affect app functionality is more like unit testing) |
➤ Kenny Hung commented: Sam (cc Tammy YangScott Yan ) v230711-ionic (0.82.0) QA found the [issue] User can’t register asset when using capture camera. ( https://app.asana.com/0/1201083422707776/1205006460193938/f ) please re-release. |
➤ Kenny Hung commented: Sam Just confirm, will you re-release today? |
➤ Sam commented: Kenny Hung submitted re-released ( https://dt42-numbers.slack.com/archives/CUSDE44MD/p1689237823315869 )with fix containing ✓ [issue] User can’t register asset when using capture camera. ( https://app.asana.com/0/0/1205006460193938 ) |
➤ Kenny Hung commented: Sam(cc Tammy YangSofia Yan) I want to check about Appsflyer implement I notice seems the app doesn’t track the download on appsflyer from 0.81.3. Because I perform SDK integration test, nothing happened. Therefore, I want to check with you.
The potential risk should be growth part: If we want to run some paid media (i.e. facebook ads.) for next 1-2 months, it need to wait for Appsflyer fixed. |
➤ Sam commented: Kenny Hung we migrated to capacitor 5 but officiall appsflyers plugin didn't migrated yet I explained more [Workaround] appsflyer migration ( https://app.asana.com/0/1201016280880500/1204897558131123 ) before. So we need to wait. I would chose option 2. Let me know what you think? |
➤ Kenny Hung commented: Sam (cc Tammy YangSofia Yan) oh...I haven't seen this task.😅 I suggest you could help to add like this message on limitation of release note. Let QA, PM, other member know this affect. Okay, we could continue testing(choose #2). |
➤ Kenny Hung commented: Sam (cc Tammy YangScott Yan) QA found [issue] can't register new account on some android device ( https://app.asana.com/0/1201083422707776/1205042457187289/f ), QA guess it maybe relate about android version. It must be fixed before we publish this version. QA worried about there may be more potential issues on v230711-ionic, QA will test whole test items & let Sam could fix together. Therefore, this version will be fail & QA will keep going to test this version. How do you think? |
➤ Tammy Yang commented: Just wondering, if it is related to the android version, does our live version has the same issue? If yes, we need to fix the issue asap but there is no reason to fail the release. If not, it is not an android-version issue but an issue on the release and it's fair to fail. |
➤ Kenny Hung commented: Tammy Yang Thanks for clarifying. No, current public version 0.81.3/0.79.0 is normal, only found on this version(0.82.x) |
➤ Sam commented: Kenny Hung thats unexpected [issue] can't register new account on some android device ( https://app.asana.com/0/1201083422707776/1205042457187289 ) no code related to sign up was changed i will have a look and see why sign up is not working. During testing i just did sign in and didn’t test sign up. Will test from now on. |
➤ Sam commented: Kenny Hung (cc: Scott Yan) Release reminder submitted ( https://dt42-numbers.slack.com/archives/CUSDE44MD/p1689701882439349 ). Contains fix for [issue] can't register new account on some android device ( https://app.asana.com/0/1201083422707776/1205042457187289 ). Tested on
|
➤ Kenny Hung commented: Sam Please help to send Redmi Note 10s back to QA. |
➤ Sam commented: Kenny Hung, release reminder submitted v230711-capture-app-ionic-launch (postpone from 0627) [0.82.4] official release ( https://dt42-numbers.slack.com/archives/CUSDE44MD/p1690878840466929 ) |
➤ Kenny Hung commented: Sam Could you help to confirm, what is the minimum iOS of this version? I could install 0.82.4 on iOS 15.2. |
➤ Sam commented: Kenny Hung, yes you are correct. I checked PR for Feat set ios minimum version to 15.7 ( #2916 ) was not merged to 0.82.4. And I think I know why. After code review if it's marked as done from asana side it will close Feat set ios minimum version to 15.7 ( #2916 ) without merging. I think next time I should carefully check what is merged and what is not. I would recommend
Let me know what you think. |
➤ Kenny Hung commented: Sam It's okay, I just want to clarify the current status.😆 What is the iOS minimum version of 0.82.4? I check testfilght, it shows iOS 13. |
➤ Sam commented: Kenny Hung yes correct it's 13.0 [0.82.4] |
➤ Kenny Hung commented: Sam (cc Tammy YangScott YanSherry Chung ) ionic-v230711 official release conditional pass issue summary
|
﹍Product code:﹍
﹍Release milestone:﹍
﹍Testing level:﹍
﹍Release note:﹍
﹍Expectation:﹍
﹍Known Issue:﹍
﹍Limitation:﹍
﹍Test Link﹍:
﹍Note:﹍
﹍Testing Results (add this after QA test)﹍:
﹍Testing Plan (add this after QA test):﹍
Testing Levels:
┆Issue is synchronized with this Asana Milestone by Unito
┆Created By: Kenny Hung
The text was updated successfully, but these errors were encountered: