numbersprotocol / capture-lite

A photo-sharing app with only verifiable photos and videos.
https://numbersprotocol.github.io/
GNU General Public License v3.0
25 stars 6 forks source link

v230321-capture-app-ionic #2650

Closed sync-by-unito[bot] closed 1 year ago

sync-by-unito[bot] commented 1 year ago

﹍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 task by Unito ┆Created By: Tammy Yang

sync-by-unito[bot] commented 1 year ago

➤ Kenny Hung commented:

Sam (cc Tammy YangScott Yan) According to discussion ( https://dt42-numbers.slack.com/archives/C0323488MEJ/p1679398858211379?thread_ts=1679395488.564919&cid=C0323488MEJ ), when [FR] asset saved to device camera roll (Can save the Captures to local) ( https://app.asana.com/0/1203982782913024/1203277148217555 ) is completed, please release ionic-v230321 About [Code Quality] refactor src/app/features/home/custom-camera/pre-publish-mode/pre-publish-mode.component.ts according ChatGPT's recommendation ( https://app.asana.com/0/1203982782913024/1204028865047890 ) may could move to next sprint.

sync-by-unito[bot] commented 1 year ago

➤ Kenny Hung commented:

Sam (cc Tammy YangScott YanSherry Chung)

v230321(0.78.0) the issue summary

  1. In this version, Oppo will take a whole black photo sometimes. For testing, QA take 20 photos will get 8 black photos. Comment by @Kenny Hung on [FR] asset saved to device camera roll (Can save the Captures to local) ( https://app.asana.com/0/0/1203277148217555/1204244833150871/f )
  2. Android device can't save the video asset on the device.
  3. unupload page need fixed: Comment by @Kenny Hung on [FR] Should be able to view photos even if the registration fails ( https://app.asana.com/0/0/1203921730220982/1204245459567168/f )
    1. force text-alight to right
    2. for options button I think only show the "remove".
    3. for user name, should let it on the middle

Please arrange re-release before next Monday EOD.

sync-by-unito[bot] commented 1 year ago

➤ Sam commented:

Kenny Hung, understood.

  1. I might need Oppo to debug in development mode.
  2. It seems like Oppo related issue, as mentioned here ( https://app.asana.com/0/0/1203277148217555/1204250033576996/f ) on my android (Galaxy M12) I can take photo/video and save to gallery. If I can get my hands on Oppo device I can better debug and solve this issue.
  3. Agree need to forcefully override text alignment

Yes I can re-release before next Monday 🙏

sync-by-unito[bot] commented 1 year ago

➤ Kenny Hung commented:

Updated for #1

QA can't reproduce it today, Sam when you get the Oppo, please help try to reproduce it, if still can't reproduce, please let QA know & QA will track this issue in the future.

sync-by-unito[bot] commented 1 year ago

➤ Sam commented:

Firebase release submitted ( https://dt42-numbers.slack.com/archives/CUSDE44MD/p1680238331565629 )