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

feat : Account Vanishing #6098

Merged
merged 5 commits into from
Jan 9, 2025
Merged

Conversation

neeldoshii
Copy link
Contributor

@neeldoshii neeldoshii commented Jan 4, 2025

Fixes

Description

Caution

Don't enter your main account credentials while testing the account vanishing feature.

Steps to follow:-

  1. Be loggedin at wikimedia and then create public account using this link
  2. Make sure your username is more like youroriginalusername_test1. for instance your main username is hello then create using hello_test1
  3. Reason for Creating new account (paste this)
    Creating a test account for testing account deletion (New Google Play policy about account deletion #5202) original account is {your username}

Steps to delete via app:-
Click on hamburger menu -> go to settings -> Scroll Down -> Click on Vanish Account.

Room for more improvement(once this merged)

  1. Show toast once the account is vanished.
  2. Log the user out, same way we perform the logout.
  3. Change the theme of AlertDialog to support Material3

🌟 h/to thanks @dbrant for this PR. Helped a lot.
🌟 h/to thanks @whym for helping creating multiple accounts :)

Have squashed merge the commits for ease in merging.

Video

delete.mp4

@@ -11,7 +10,6 @@ import android.net.Uri
import android.os.Bundle
import android.text.Editable
import android.text.TextWatcher
import android.view.KeyEvent
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For maintainers:
I noticed there were unused imports so I have removed this two above imports.

@neeldoshii neeldoshii mentioned this pull request Jan 4, 2025
@nicolas-raoul

This comment was marked as outdated.

<string name="account">Account</string>
<string name="vanish_account">Vanish Account</string>
<string name="account_vanish_request_confirm_title">Vanish account warning</string>
<string name="account_vanish_request_confirm"><![CDATA[Vanishing is a <b>last resort</b> and should <b>only be used when you wish to stop editing forever</b> and also to hide as many of your past associations as possible.<br/><br/>Account deletion on Wikipedia is done by changing your account name to make it so others cannot recognize your contributions in a process called account vanishing. <b>Vanishing does not guarantee complete anonymity or remove contributions to the projects</b>.]]></string>
Copy link
Collaborator

Choose a reason for hiding this comment

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

It should say "Account deletion on Wikimedia Commons".

Copy link
Member

Choose a reason for hiding this comment

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

Great catch, thanks!

@whym
Copy link
Collaborator

whym commented Jan 9, 2025

More on the text (maybe to be discussed later): I think the dialog text can be shorter (for the typically small space of the dialog), partly because the user will read it later anyway in the web view in the current workflow.

Instead we might want to include a link to read more about vanishing, besides "Cancel" and "Vanish account". Link: https://meta.wikimedia.org/wiki/Account_vanishing
(Or https://www.mediawiki.org/wiki/Help:Extension:CentralAuth/Global_Vanishing ) ?

@neeldoshii neeldoshii marked this pull request as draft January 9, 2025 13:03
@neeldoshii neeldoshii marked this pull request as ready for review January 9, 2025 13:18
@neeldoshii
Copy link
Contributor Author

Nice catch whym! I directly copied the string from Wikipedia app.
I have added the comment for SingleWebViewActivity and also thanks Nicolas for fixing the merge conflict before :)

@neeldoshii neeldoshii requested a review from whym January 9, 2025 13:20
@nicolas-raoul
Copy link
Member

@whym Would you mind creating a separate issue about #6098 (comment)? The high priority for now is to get this feature released as soon as possible for policy reasons, a second phase can focus on UX improvements. :-) Thanks a lot!

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.

New Google Play policy about account deletion
3 participants