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

Exclude empty pages from the list of outdated pages #2660

Closed
Tracked by #133 ...
MizukiTemma opened this issue Feb 15, 2024 · 4 comments · Fixed by #2735
Closed
Tracked by #133 ...

Exclude empty pages from the list of outdated pages #2660

MizukiTemma opened this issue Feb 15, 2024 · 4 comments · Fixed by #2735
Assignees
Labels
effort: low Should be doable in <4h enhancement This improves an existing feature prio: high Needs to be resolved ASAP.
Milestone

Comments

@MizukiTemma
Copy link
Member

MizukiTemma commented Feb 15, 2024

Motivation

We currently show all the pages that are not updated more than 1 year in "Outdated pages" section in the dashboard.

However, the date of last modification is not important for empty pages, because they exist to function as "paragraph title".

⚠️ Keep loading time in mind. It might be a problem for regions with a lot of pages.

Proposed Solution

Exclude empty pags from the list of outdated pages.

Alternatives

Leave it as it is.

User Story

Additional Context

We want to encourage users to keep the contents actual. (OKR)
We want to increase usability of Integreat System by showing only non empty pages, which really needs updating. (OKR)

Design Requirements

@MizukiTemma MizukiTemma added the feature New feature or request label Feb 15, 2024
This was referenced Feb 15, 2024
@MizukiTemma MizukiTemma added this to the 24Q1 milestone Feb 19, 2024
@MizukiTemma
Copy link
Member Author

MizukiTemma commented Feb 19, 2024

This issue has been moved to the actual milestone because it needs to be implemented in an earlier stage of this year so we can count how many pages "really" must be updated (OKR).

@MizukiTemma MizukiTemma added the prio: medium Should be scheduled in the forseeable future. label Feb 19, 2024
@JoeyStk JoeyStk added bug Something isn't working and removed feature New feature or request labels Feb 27, 2024
@JoeyStk
Copy link
Contributor

JoeyStk commented Feb 27, 2024

Currently it's not possible to exclude empty pages from a search query in the page tree, right? Am I understanding it correctly then that implementing this filter would also be part of this issue? :)

@JoeyStk JoeyStk added enhancement This improves an existing feature and removed bug Something isn't working labels Feb 27, 2024
@JoeyStk JoeyStk modified the milestones: 24Q1, 24Q2 Feb 27, 2024
@JoeyStk JoeyStk added question Further information is requested effort: low Should be doable in <4h labels Feb 27, 2024
@timobrembeck
Copy link
Member

@JoeyStk yes I agree, the dashboard section about outdated pages should not only contain the correct number of pages, but should also link to the page tree without the empty pages, and for the link to work, we need this filter.

@JoeyStk JoeyStk self-assigned this Feb 28, 2024
@JoeyStk JoeyStk removed the question Further information is requested label Feb 28, 2024
@osmers
Copy link

osmers commented Mar 12, 2024

Could I add that it would make sense to only show pages that are visible in the app? So I think filter by publication status, either only published, or all but draft?

Not sure how we handle automatic saves - or rather, whether it makes sense to include them or not.
I think maybe we shouldn't include them there bcs we already have a section for automatic saves and it is highly unlikely that we'll have many pages which are automatic saves and outdated, right?

So maybe set the filter to only show outdated pages with content that are published? I noticed this issue bcs here https://admin.integreat-app.de/lkvechta/ when you click "Zur Seite" for outdated pages, you can see that the page is in draft.

@JoeyStk JoeyStk added prio: high Needs to be resolved ASAP. and removed prio: medium Should be scheduled in the forseeable future. labels Apr 3, 2024
@JoeyStk JoeyStk modified the milestones: 24Q1, 24Q2 Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort: low Should be doable in <4h enhancement This improves an existing feature prio: high Needs to be resolved ASAP.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants