[full-ci] refactor pagination into composable, cleanup store, views and tests

[fschade]

Description

See #6048

Related Issue

• Part of Cleanup implicit dependencies #6048

Motivation and Context

clean code ™️

How Has This Been Tested?

• unit tests
• integration tests

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests

Checklist:

• Code changes
• Unit tests added

[update-docs]

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

[ownclouders]

Results for oC10Basic https://drone.owncloud.com/owncloud/web/20505/12/1
💥 The acceptance tests pipeline failed. The build has been cancelled.

[ownclouders]

Results for oC10SharingAccept https://drone.owncloud.com/owncloud/web/20518/16/1
The following scenarios passed on retry:

• webUISharingAcceptSharesToRoot/acceptShares.feature:108

[ownclouders]

Results for oC10Files1 https://drone.owncloud.com/owncloud/web/20518/17/1
💥 The acceptance tests pipeline failed. The build has been cancelled.

[kulmann]

The duplication of pagination templates is a bit verbose. Could still live in a dedicated pagination component with a few props. Would be more or less a template-only component. v-if should be placed in the views like you have it now, not inside the component.

Please decide for yourself if you want to do that in this PR or in a separate one to get this awesome change going fast. 🚀

[fschade]

The duplication of pagination templates is a bit verbose. Could still live in a dedicated pagination component with a few props. Would be more or less a template-only component. v-if should be placed in the views like you have it now, not inside the component.

Please decide for yourself if you want to do that in this PR or in a separate one to get this awesome change going fast. 🚀

good point, done expect the outer if decision, i think in the case of this component where a pages of 0 is a potential displaying glitch its ok to have it battery included inside the component and let it decide on its own what to do.

THANKS for your review ❤️

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

67.9% Coverage
21.4% Duplication

[kulmann]

😍