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 status pages #107

Merged
merged 2 commits into from
Nov 13, 2022
Merged

Add status pages #107

merged 2 commits into from
Nov 13, 2022

Conversation

Schmiddiii
Copy link
Collaborator

Licensing

  • I confirm that this is either my code or was released under the terms of a GPLv3-or-later compatible license. Also I agree to release it here under the terms of the GPLv3-or-later.

Description

Had a little bit of time and implemented showing status pages if a page was empty. @TheEvilSkeleton if you have time feel free to review. The texts and iconds should probably be discussed a little more.

E.g. watch-later page:

tubefeeder-status-page

Linked Issue

Related to #95.

Copy link
Contributor

@TheEvilSkeleton TheEvilSkeleton left a comment

Choose a reason for hiding this comment

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

Looking good!

Only one suggestion, and then everything should be good to go.

data/resources/ui/watch_later.ui Outdated Show resolved Hide resolved
Co-authored-by: Hari Rana <theevilskeleton@riseup.net>
@Schmiddiii Schmiddiii merged commit a294dab into master Nov 13, 2022
@Schmiddiii Schmiddiii deleted the status-pages branch November 13, 2022 10:05
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.

None yet

2 participants