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

Convert AccountAuthenticator to Kotlin #12126

Closed

Conversation

alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Nov 2, 2023

  • Tests written, or not not needed

@tobiasKaminsky
Copy link
Member

Detekt is failing.
Restarted Drone.

Copy link
Member

@tobiasKaminsky tobiasKaminsky left a comment

Choose a reason for hiding this comment

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

Fix detekt

@alperozturk96
Copy link
Collaborator Author

/rebase

@tobiasKaminsky
Copy link
Member

/rebase

@nextcloud-command nextcloud-command force-pushed the refactor/convert-AccountAuthenticator-to-kt branch from d543f2a to e53f8e2 Compare December 3, 2023 08:19
@alperozturk96 alperozturk96 force-pushed the refactor/convert-AccountAuthenticator-to-kt branch from e53f8e2 to b549906 Compare February 22, 2024 08:25
@alperozturk96 alperozturk96 force-pushed the refactor/convert-AccountAuthenticator-to-kt branch from b549906 to 944f48f Compare March 14, 2024 08:22
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
@alperozturk96 alperozturk96 force-pushed the refactor/convert-AccountAuthenticator-to-kt branch from 944f48f to 2ec36ca Compare March 15, 2024 08:19
Copy link

Codacy

Lint

TypemasterPR
Warnings7171
Errors33

SpotBugs

CategoryBaseNew
Bad practice6868
Correctness6969
Dodgy code351351
Experimental22
Internationalization77
Malicious code vulnerability22
Multithreaded correctness66
Performance5656
Security1919
Total580580

Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/12126.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

Copy link

@alperozturk96
Copy link
Collaborator Author

Out of date due to conflicts

@tobiasKaminsky tobiasKaminsky deleted the refactor/convert-AccountAuthenticator-to-kt branch October 7, 2024 05:28
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.

2 participants