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

Fix nominate for deletion #6194

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

parneet-guraya
Copy link
Contributor

@parneet-guraya parneet-guraya commented Feb 18, 2025

Fixes #6192

What changes did you make and why?

  • put a condition to not fetch achievements in beta variant.
  • passed callback to fragment to trigger refresh after nominate to delete call finishes.

Tests performed (required)
Tested on both beta and prod.
Device: Oneplust 9RT 5G, Android 14

Record_2025-02-18-20-14-20.1.mp4

Signed-off-by: parneet-guraya <[email protected]>
Copy link
Member

@nicolas-raoul nicolas-raoul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing this branch in prod now.
Like before, it spins forever after deletion, and the nomination is not actually sent.
Would you mind trying?

@parneet-guraya
Copy link
Contributor Author

parneet-guraya commented Feb 19, 2025

Strange! It does work for me. Could you try going back and opening the same media detail again? Perhaps there might be issues with refreshing? Did you see the toast message when clicking?

Record_2025-02-19-12-50-25.mp4

@nicolas-raoul
Copy link
Member

Is your screencast in beta or prod flavor?

@parneet-guraya
Copy link
Contributor Author

In prod

Record_2025-02-19-13-11-41.mp4

@nicolas-raoul
Copy link
Member

Would you mind trying on a bad contribution in Explore's "Uploaded via mobile", for instance the recent upload called linkinpng?

@nicolas-raoul
Copy link
Member

Screenshot_20250219-171851.png

@parneet-guraya
Copy link
Contributor Author

Would you mind trying on a bad contribution in Explore's "Uploaded via mobile", for instance the recent upload called linkinpng?

Sure!

document_6174842166125270165.mp4

@parneet-guraya
Copy link
Contributor Author

Okay, got the bug! So, basically if you see some contributions in the list doesn't have a username showing. For some reason it isn't stored (should I create another ticket for this?). Basically, when making the request for deletion nomination userName is also passed but here nothing to pass, so request fails. That's why it never refreshes, although I should mention that the picture still get nominated eventually (if we go back and open the media page again), maybe the request that fails is part of chain and it doesn't affect the actual deletion request just that our flow of request doesn't return as expected so no refresh is triggered.

Error in the logs when username is empty so request goes out with title=User_talk%3A

Screenshot from 2025-02-19 17-57-56

document_6174842166125270324.mp4

In this case username is actually i.e User_talk%3AParryguraya appended to it.

Screenshot from 2025-02-19 17-50-56

document_6174842166125270327.mp4

I think I need to fix the other issue first then this can be merged?

@nicolas-raoul
Copy link
Member

Other issue sounds good, thanks! :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nominate for deletion: spinning forever, no effect
2 participants