-
-
Notifications
You must be signed in to change notification settings - Fork 756
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
Settings page #1612
Settings page #1612
Conversation
@@ -58,3 +58,4 @@ security: | |||
- { path: ^/resetting, role: IS_AUTHENTICATED_ANONYMOUSLY } | |||
- { path: /(unread|starred|archive).xml$, roles: IS_AUTHENTICATED_ANONYMOUSLY } | |||
- { path: ^/, roles: ROLE_USER } | |||
- { path: ^/settings, roles: ROLE_SUPER_ADMIN } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With my ROLE_USER user, I can access to this URL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yeah I haven't tested that case. It's done.
90c4dfd
to
c9c6aef
Compare
@@ -41,25 +47,6 @@ wallabag_import: | |||
twig: | |||
debug: "%kernel.debug%" | |||
strict_variables: "%kernel.debug%" | |||
globals: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice removal :-)
Still conflicts due to the last merged PR. |
dbbb20c
to
58a772a
Compare
Don't forget to update parameters according to #1629. :) |
These default configuration value shouldn't be in parameters.yml.
We'll remove it later, on the stable release
Warn user if pocket_consumer_key isn't defined
58a772a
to
48e3007
Compare
@tcitworld updated. |
You deleted the |
@tcitworld It's sent to the template here and I don't have to update the template because the |
export_txt: Activer l'export TXT | ||
export_xml: Activer l'export XML | ||
pocket_consumer_key: Clé d'authentification Pocket pour importer les données (https://getpocket.com/developer/docs/authentication) | ||
shaarli_url: URL de Shaarli, si le service Diaspora est activé |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
si le service Diaspora est activé
> si le service est activé
Fix #1575
This PR add an internal config page where admin can configure its wallabag instance. Most of the parameters from the
parameters.yml.dist
were moved to this page as they can be switched during run time.To achieve that I've used the CraueConfigBundle. Each parameter has a section which allow to have a less disgusting internal config page.
This page is only accessible by user with role
ROLE_SUPER_ADMIN
.This table is filled using the InstallCommand (at the same time we build the user config).
Things to check before validate this PR:
Preview:
Bonus
I've added a message on the Pocket import page when the consumer key isn't defined: