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

Add explicit favorite capability entry #37673

Merged
merged 2 commits into from
Jul 9, 2020
Merged

Add explicit favorite capability entry #37673

merged 2 commits into from
Jul 9, 2020

Conversation

PVince81
Copy link
Contributor

@PVince81 PVince81 commented Jul 9, 2020

Description

Add explicit favorite capability entry
As it's already supported in OC 10, it's set to true.

Related Issue

For owncloud/ocis#354

Motivation and Context

This will make it possible for ocis-web to decide whether to enable the feature in the UI or not.
As this is not implemented yet in OCIS, the feature/capability will be disabled there.

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

@LukasHirt FYI

@PVince81
Copy link
Contributor Author

PVince81 commented Jul 9, 2020

@micbar FYI

this means that Phoenix running against OC <= 10.5 will not display the favorites feature

As it's already supported in OC 10, it's set to true.
@PVince81
Copy link
Contributor Author

PVince81 commented Jul 9, 2020

added changelog entry

@PVince81 PVince81 force-pushed the favorite-capability branch from 06f6c97 to 06d486c Compare July 9, 2020 09:16
Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

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

Could add an acceptance test scenario like https://github.com/owncloud/core/blob/master/tests/acceptance/features/apiCapabilities/capabilities.feature#L80

Test for the new capability, and @skipOnOcV10.3 @skipOnOcV10.4 @skipOnOcV10.5

@owncloud owncloud deleted a comment from update-docs bot Jul 9, 2020
@PVince81
Copy link
Contributor Author

PVince81 commented Jul 9, 2020

not sure if there is much value in such test

@PVince81
Copy link
Contributor Author

PVince81 commented Jul 9, 2020

it's green, please approve

@codecov
Copy link

codecov bot commented Jul 9, 2020

Codecov Report

Merging #37673 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #37673   +/-   ##
=========================================
  Coverage     64.74%   64.74%           
  Complexity    19363    19363           
=========================================
  Files          1281     1281           
  Lines         75628    75628           
  Branches       1333     1333           
=========================================
  Hits          48966    48966           
  Misses        26270    26270           
  Partials        392      392           
Flag Coverage Δ Complexity Δ
#javascript 54.03% <ø> (ø) 0.00 <ø> (ø)
#phpunit 65.93% <ø> (ø) 19363.00 <ø> (ø)
Impacted Files Coverage Δ Complexity Δ
apps/files/lib/Capabilities.php 100.00% <ø> (ø) 2.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5574a5a...0cfbdfc. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Jul 9, 2020

Codecov Report

Merging #37673 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #37673   +/-   ##
=========================================
  Coverage     64.74%   64.74%           
  Complexity    19363    19363           
=========================================
  Files          1281     1281           
  Lines         75628    75628           
  Branches       1333     1333           
=========================================
  Hits          48966    48966           
  Misses        26270    26270           
  Partials        392      392           
Flag Coverage Δ Complexity Δ
#javascript 54.03% <ø> (ø) 0.00 <ø> (ø)
#phpunit 65.93% <ø> (ø) 19363.00 <ø> (ø)
Impacted Files Coverage Δ Complexity Δ
apps/files/lib/Capabilities.php 100.00% <ø> (ø) 2.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5574a5a...0cfbdfc. Read the comment docs.

@PVince81 PVince81 merged commit bb2ddd7 into master Jul 9, 2020
@delete-merged-branch delete-merged-branch bot deleted the favorite-capability branch July 9, 2020 14:54
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.

3 participants