Skip to content

Conversation

@AndyScherzinger
Copy link
Member

Resolves #4774

  • correcting all menu file names and
  • adding the corresponding guidelines chapter to CONTRIBUTING.md including a reference to the original Ribot guidelines.

@nextcloud-android-bot
Copy link
Collaborator

@codecov
Copy link

codecov bot commented Nov 3, 2019

Codecov Report

Merging #4789 into master will decrease coverage by 0.04%.
The diff coverage is 9.09%.

@@             Coverage Diff              @@
##             master    #4789      +/-   ##
============================================
- Coverage     17.57%   17.52%   -0.05%     
  Complexity        3        3              
============================================
  Files           377      377              
  Lines         32403    32400       -3     
  Branches       4572     4572              
============================================
- Hits           5694     5678      -16     
- Misses        25799    25803       +4     
- Partials        910      919       +9
Impacted Files Coverage Δ Complexity Δ
...ncloud/android/ui/activity/UploadListActivity.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...extcloud/client/errorhandling/ShowErrorActivity.kt 0% <0%> (ø) 0 <0> (ø) ⬇️
...i/fragment/contactsbackup/ContactListFragment.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...ncloud/android/ui/preview/PreviewTextFragment.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...owncloud/android/ui/trashbin/TrashbinActivity.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...android/ui/fragment/FileDetailSharingFragment.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...owncloud/android/ui/activity/UserInfoActivity.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...cloud/android/ui/preview/PreviewMediaFragment.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...m/owncloud/android/ui/adapter/UserListAdapter.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...ncloud/android/ui/fragment/FileDetailFragment.java 0% <0%> (ø) 0 <0> (ø) ⬇️
... and 15 more

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.

Some smaller changes

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@AndyScherzinger
Copy link
Member Author

@tobiasKaminsky please re-review. All requested changed have been done 😃 Thanks for the review ❤️

@AndyScherzinger AndyScherzinger dismissed tobiasKaminsky’s stale review November 13, 2019 10:48

All changes have been made as per request

@nextcloud-android-bot
Copy link
Collaborator

@nextcloud-android-bot
Copy link
Collaborator

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/11607.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.

@nextcloud-android-bot
Copy link
Collaborator

Codacy

469

Lint

TypemasterPR
Warnings5959
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings119
Security Warnings44
Dodgy code Warnings138
Total423

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings119
Security Warnings44
Dodgy code Warnings138
Total423

@tobiasKaminsky tobiasKaminsky merged commit dbe12e6 into master Nov 18, 2019
@delete-merged-branch delete-merged-branch bot deleted the name_pattern_menus branch November 18, 2019 14:26
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.10.0 milestone Nov 18, 2019
tobiasKaminsky added a commit that referenced this pull request Nov 19, 2019
dbe12e6 Merge pull request #4789 from nextcloud/name_pattern_menus
77b6b17 Merge pull request #4841 from nextcloud/disableOldLoginMethod
b561f68 allow to switch off old login method
b59d96a [tx-robot] updated from transifex
9ae5507 [tx-robot] updated from transifex
bfdaba8 [tx-robot] updated from transifex
c82c080 daily dev 20191116
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.

Improve menu file name pattern

4 participants