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

Delete all accounts on first run #949

Merged
merged 1 commit into from
May 4, 2017
Merged

Conversation

mario
Copy link
Contributor

@mario mario commented May 3, 2017

This fixes #820

Copy link
Member

@AndyScherzinger AndyScherzinger left a comment

Choose a reason for hiding this comment

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

👍 nice trick :)

@AndyScherzinger
Copy link
Member

AndyScherzinger commented May 3, 2017

👍

Approved with PullApprove

@mario
Copy link
Contributor Author

mario commented May 3, 2017

Glad you like it :)

@tobiasKaminsky
Copy link
Member

tobiasKaminsky commented May 4, 2017

👍

Approved with PullApprove

@tobiasKaminsky tobiasKaminsky merged commit 54eba41 into master May 4, 2017
@tobiasKaminsky tobiasKaminsky deleted the fix-crash-on-login branch May 4, 2017 09:17
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 1.5.0 milestone May 4, 2017
@atomspring
Copy link

atomspring commented Oct 14, 2017

This is still happening to me, where AccountManagerService is being skipped since the account already exists... but NextCloud isn't listed in Android's settings>>Accounts pane.

I went into /data/system_de/accounts_de.db using sqlite3 and noted that the only item listing a nextcloud account was in meta, a .dump of meta lists this line: INSERT INTO "meta" VALUES('auth_uid_for_type:nextcloud','10145');

I deleted this line, did a soft-reboot of my device and tried adding my account using the token generated on the web UI, logcat still shows AccountManagerService warning about adding the account because it exists using the insertAccountIntoDatabase function. Where else could the accounts be stored?

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.

Crash on login
4 participants