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

wip: Add advanced source settings #1461

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

wip: Add advanced source settings #1461

wants to merge 2 commits into from

Conversation

jtojnar
Copy link
Member

@jtojnar jtojnar commented Jul 30, 2023

Most people will not use filter for most sources so let’s introduce advanced settings and only show filter when filled in or when advanced settings are enabled.

This will allow us to implement less common features like archival, reduced update frequency or icon overriding.

WIP because I do not like the the toggle button placement.

Also introduce “Icon override” proof of concept (only client side for now). At minimum we will want to allow URL entry but upload would be even better.

Most people will not use filter for most sources so let’s only show it
when filled in or when advanced settings are enabled.

With advanced settings we will be able to implement less common features
like archival, reduced update frequency or icon overriding.

WIP because I do not like the the toggle button placement.
Currently only client side is implemented.

At minimum we will want to allow URL entry but upload would be even better.
@netlify
Copy link

netlify bot commented Jul 30, 2023

Deploy Preview for selfoss canceled.

Name Link
🔨 Latest commit 86a2650
🔍 Latest deploy log https://app.netlify.com/sites/selfoss/deploys/64c63193343c5900085a0a7e

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.

1 participant