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

Disable reconfiguring in GD projects #5705

Merged
merged 1 commit into from
Aug 25, 2023

Conversation

grzesiek2010
Copy link
Member

@grzesiek2010 grzesiek2010 commented Aug 17, 2023

Closes #5704

What has been done to verify that this works as intended?

I've tested the fix manually and added automated tests.

Why is this the best possible solution? Were any other approaches considered?

We decided that disabling reconfiguring for GD projects would be a good solution for GD projects that still exist to avoid loosing data.

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

Reconfiguring projects should be disabled (not visible) for GD projects. It's a safe change so verifying that the option is visible for non-GD projects nd hidden for GD projects would be enough.

Do we need any specific form for testing your changes? If so, please attach one.

No.

Does this change require updates to documentation? If so, please file an issue here and include the link below.

No.

Before submitting this PR, please make sure you have:

  • run ./gradlew checkAll and confirmed all checks still pass OR confirm CircleCI build passes and run ./gradlew connectedDebugAndroidTest locally.
  • verified that any code or assets from external sources are properly credited in comments and/or in the about file.
  • verified that any new UI elements use theme colors. UI Components Style guidelines

@grzesiek2010 grzesiek2010 marked this pull request as ready for review August 17, 2023 14:36
@dbemke
Copy link

dbemke commented Aug 25, 2023

Tested with Success!

Verified on device with Android 10

Verified cases:

  • disabled reconfiguring projects in Google Drive project
  • reconfiguring via QR code in Central, Aggregate projects

The PR was tested on selfsigned build of the apk, it will be tested again in the final build (with upgrades and GD projects) before release.

@srujner
Copy link

srujner commented Aug 25, 2023

Tested with Success!

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

Successfully merging this pull request may close these issues.

Google Drive server settings changed after reconfiguring via QR code
4 participants