-
Notifications
You must be signed in to change notification settings - Fork 6
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
[issue] When I reload “VERIFIED” tab from Capture Cam, it keeps loading many many very old Captures (seems like it tried to reload everything instead of only the latest ones) #3057
Comments
➤ Kenny Hung commented: Scott Yan (cc Tammy YangBofu Chen) This task I'll suggest arranging into 1023 sprint. |
➤ Sam commented: Kenny Hung yes that’s a current implementation when you pull it will refetch all. And of course it make sense to reload only first 10 latest items. One more thing to mention why old implementation is pulling all the assets might be that Before registering it check if the capture was uploaded before if yes then it stop uploading to backend. So currently it check for duplicates in ionic side. |
➤ Kenny Hung commented: Added https://docs.google.com/presentation/d/1jdmYyUynCzaMsfHMdAdzJ23KnF5XhgsJt27FgPjNQiY/edit#slide=id.p ( https://docs.google.com/presentation/d/1jdmYyUynCzaMsfHMdAdzJ23KnF5XhgsJt27FgPjNQiY/edit#slide=id.p ) in the description. |
➤ Kenny Hung commented: According to https://dt42-numbers.slack.com/archives/C0323488MEJ/p1698742854255449 ( https://dt42-numbers.slack.com/archives/C0323488MEJ/p1698742854255449 ) push to 1106 sprint. |
➤ Sam commented: Kenny Hung, I'm still working on this task. I also found that when I reload it do too many request then it should do which also contributes to slow reloads. I need more time to work on this task. Today I will focus on this task and later will notify with updates if I have any. |
➤ Kenny Hung commented: Sam (cc Bofu ChenJames ChienScott YanTammy Yang) Just confirm, since the goal of this task is to enable the reload function to reload the current latest captures, it's necessary first to address the issue of excessive API calls during assets fetch to complete this task, right? |
➤ Sam commented: Kenny Hung, I found the root cause (explained in video). Solution for excessive API calls is to
|
➤ Sam commented: Kenny Hung, regarding your question ( https://app.asana.com/0/0/1205627319931691/1206137299959857/f ). it's necessary first to address the issue of excessive API calls during assets fetch to complete this task, right? reload function to reload the current latest capturesis not blocked or depends on issue of excessive API calls during assets fetch. I will create separate asana card with high priority and make sure we arrange as soon as possible. |
➤ Sam commented: Kenny Hung, asana card is created [Issue] Excessive API calls during reload/restore captures ( https://app.asana.com/0/1201016280880500/1206137436792270/f ) |
➤ Kenny Hung commented: Sam (cc Bofu ChenJames ChienScott YanTammy Yang) Thanks, please focus on this task unless other tasks could help to improve or block this task. |
➤ Sam commented: Kenny Hung, just status update I finished lazy load for VERIFIED tab only here is the demo. Next I will work on lazy load COLLECTED tab. After that I will test few more time and prepare firebase release. I plan to submit release reminder for Tue (Dec 12) morning. |
➤ Kenny Hung commented: It seems that every time user scroll down, the latest 10 assets are added, am I right? |
➤ Sam commented: Almost. PUBLIC CAPTURE CAM when user does pull to refresh it does the following
When use scroll down
VERSION IN DEMO (new implementation) when user does pull to refresh 25 things happen
When user scroll down
Of course new implementation can be improved further and further but I think this is good enough improvement for now as you can see from demo video. |
➤ Kenny Hung commented: Sam (cc Scott YanSherry ChungTammy YangBofu ChenJames Chien) QA found when user pull down to refresh captures, that only 10 images will be loaded, and Capture Cam don’t seem to be saved. After closing and reopening the app, there are still only 10 images. The expectation should be
|
➤ Kenny Hung commented: Sam Will you re-release tomorrow? If you have certain idea to fix, maybe we could discuss with Tammy Yang. I summary the v231207 & public version on miro ( https://miro.com/app/board/o9J_knEiIYo=/?moveToWidget=3458764572931523236&cot=14 ). My suggestions are: (Needs Tammy Yang's confirmation.)
Pros & cons
Sam, if you have any better ideas, we could discuss them together tomorrow. |
➤ Sam commented: @kenny Hung sure I can release tomorrow. And we can discuss that tomorrow as well. Just make sure we confirm the changes and I can implement them |
➤ Tammy Yang commented: SamKenny Hung The requirements of the Verified page (same as the Live version)
The reason is, Capture serves as the "asset wallet" to users, so we should let users hold their own content instead of downloading only the thumbnails. Capture is not IG. This requirement made the syncing of Capture very challenging since day 1 we built it. However, solving the challenge and finding good ways to achieve it shows the value of our developers. Please propose a good plan to solve the issue. I strongly suggest Sam create a miro to explain how do we plan to solve the issue, and I want to see the flow by the end of tomorrow. Once approved, we can move on |
➤ Sherry Chung commented: Sam (cc Kenny Hung) If you need my help for further clarification. I’m available tomorrow 2:00-3:00 pm for huddle |
➤ Sam commented: Miro related to this task can be found here ( https://miro.com/app/board/o9J_knEiIYo=/?moveToWidget=3458764573039264715&cot=14 ) (Work In Progress). I will ask some question to while working on miro. Just to remind Currently in public version when user opens asset details page
2b is caused by changes related to bubble iframe. Original asset video/image suppose to be downloaded (if not locally stored) every time when user enters details page but since we use bubble iframe to show information and preview photo/videos we no longer download it locally. |
➤ Sam commented: Kenny Hung (cc: Tammy Yang, Sherry Chung) Since implementing these requirements ( https://app.asana.com/0/0/1205627319931691/1206142917957845/f ) might take some time I would recommend first fix QA feedback ( https://app.asana.com/0/0/1205627319931691/1206147430104127/f ) and release to public. After that we can focus on implementing these requirements ( https://app.asana.com/0/0/1205627319931691/1206142917957845/f ) |
➤ Sherry Chung commented: Sam (cc Kenny Hung) I would suggest we don't do release for release. We can discard current Ionic release if that didn't really solve the problem. This is the only question we need to solve in this sprint. Please try to find a solution to meet the requirement. Ping me on Slack if you need Huddle |
➤ Kenny Hung commented: Samhttps://chat.openai.com/share/6c30bc3e-2cee-4c57-a31a-7549891b6f1d ( https://chat.openai.com/share/6c30bc3e-2cee-4c57-a31a-7549891b6f1d ) I'm not sure if this helps, but I just asked ChatGPT about possible solutions, which might give you some inspiration. |
➤ Sam commented: Tammy Yang (cc: Sherry Chung, Kenny Hung). Capture cam public version
Capture cam public version
Capture cap version before adding bubble page to Details page
Kenny Hung, (cc: Tammy Yang, Sherry Chung) For now I don't have super cool solution to fix [issue] When I reload “VERIFIED” tab from Capture Cam, it keeps loading many many very old Captures (seems like it tried to reload everything instead of only the latest ones) ( https://app.asana.com/0/1201016280880500/1205627319931691/f ) however I have 1 proposal in mind When user trigger pull to refresh do the following
This proposal inform user that it will take time and make him aware. |
➤ Tammy Yang commented: I actually think this is smart, good idea! Let's do this: When users pull down to refresh, ask user
If user choose "Confirm", go to your proposal step #3 (show blocking UI with progress) If user choose "Do it later", do nothing. BTW, I hope my messages are delivered well to you, Kenny Hung and Sherry Chung When we face a super challenging issue, we need to work together like this to brainstorm and find a solution. However, how do we know the issue is super challenging? We need to communicate with each other and make sure we understand the problem well. It is important that we care about each others work and help each other. Let's go with this plan, it's not something like magic, but good enough UX to move on! |
➤ Sam commented: Kenny Hung, here is the final result of implemented proposal ( https://app.asana.com/0/0/1205627319931691/1206142917957851/f ). https://app.claap.io/numbers-protocol/feat-capture-tab-show-confirm-dialog-before-refr-c-O35CsUM4Uy-8WYI1bAHJrhA ( https://app.claap.io/numbers-protocol/feat-capture-tab-show-confirm-dialog-before-refr-c-O35CsUM4Uy-8WYI1bAHJrhA ) |
➤ Sam commented: Here is the PR ( #3171 ) |
➤ Sam commented: Here is the new PR ( #3197 ) because previously we reverted this changes ( https://app.asana.com/0/0/1206261707717918/1206483891493527/f ). |
User story
As a user of the Capture Cam feature,
I want the "VERIFIED" tab to reload and display only the latest verified captures upon refresh,
So that I can easily access the most recent content without waiting long time for refresh.
Reproduce step
Expectation
When I reload “VERIFIED” tab from Capture Cam, it should only reload the latest ones.
Additional information:
https://dt42-numbers.slack.com/archives/C0323488MEJ/p1695983857350369
https://docs.google.com/presentation/d/1jdmYyUynCzaMsfHMdAdzJ23KnF5XhgsJt27FgPjNQiY/edit#slide=id.p
┆Issue is synchronized with this Asana task by Unito
┆Created By: Kenny Hung
The text was updated successfully, but these errors were encountered: