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

[DEGGAND-88] upgrade to Android sdk 33 (13) #282

Merged
merged 11 commits into from
Aug 19, 2023

Conversation

itachi1706
Copy link
Owner

No description provided.

There are flags set for backwards compatibility reasons and should be resolved

Took 8 minutes
Fixes some changes required as we can no longer use resource (R) ids in switch cases

Took 18 minutes
Added various changes required for notifications

Took 56 minutes
@itachi1706 itachi1706 self-assigned this Aug 19, 2023
@itachi1706 itachi1706 changed the title [DEGGAND-88] upgrade to Android sdk 33 [DEGGAND-88] upgrade to Android sdk 33 (13) Aug 19, 2023
private static final String R_EGG_UNLOCK_SETTING = "egg_mode_r";

private void toastUp(String s) {
Toast toast = Toast.makeText(this, s, Toast.LENGTH_SHORT);
toast.show();
}

private static final String TAG = "NekoR";

private final ActivityResultLauncher<String> notificationPermissionLauncher =
Copy link

Choose a reason for hiding this comment

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

Identical blocks of code found in 2 locations. Consider refactoring.

@@ -35,6 +39,17 @@ private void toastUp(String s) {
toast.show();
}

private static final String TAG = "Neko";

private final ActivityResultLauncher<String> notificationPermissionLauncher =
Copy link

Choose a reason for hiding this comment

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

Identical blocks of code found in 2 locations. Consider refactoring.

@codeclimate
Copy link

codeclimate bot commented Aug 19, 2023

Code Climate has analyzed commit 66e022f and detected 0 issues on this pull request.

View more on Code Climate.

@sonarcloud
Copy link

sonarcloud bot commented Aug 19, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

0.0% 0.0% Coverage
13.7% 13.7% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@itachi1706 itachi1706 merged commit 53bd75e into master Aug 19, 2023
15 of 16 checks passed
@itachi1706 itachi1706 deleted the DEGGAND-88-Upgrade-to-SDK-33 branch August 19, 2023 16:15
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.

1 participant