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

Tableau de bord / page d'accueil configurable #117

Merged
merged 9 commits into from Mar 2, 2018

Conversation

Projects
None yet
3 participants
@JocelynDelalande

JocelynDelalande commented Feb 28, 2018

Pour répondre à #84

Le code « réel » n'est pas prêt (ne pas fusionner, donc), mais j'ai déjà une proposition concernant l'interface de sélection, que je vous partage ici :

image

J'ai séparé le profil utilisateur en deux parties : identité (avec les champs qui étaient déjà présents) et paramètres d'affichage ; ce qui permet à chaque utilisateur d'avoir sa propre configuration d'affichage.

Retours bienvenus :-)

JocelynDelalande added some commits Feb 28, 2018

Add vertical space between navbar and tab content
Proximity was specially annoying in settings page.
Use clearer label for save button (IMHO)
Update might have two understandings:

- "update from the server"
- "update to the server"

« Save » is less ambiguous.
Remove redundant / inconsistent API permissions
(permission_classes already handle permissions for the whole viewset).
Slightly change the therapeut settings API logic
Before:

*/get_by_user* returned the existing therapeut settings, or empty response, that
was up to the front-end to then create (POST) a profile.

After:

*/get_by_user* returns the existing therapeut settings or creates a new default
 one on the fly if none exist.

Why?

- back-end can push default values into the profile (pave the way for next commits)
- less code
Offer user to choose which module is displayed on home/dashboard
- Record settings for that (default : all moduels enabled)
- Honor those settings: load only enabled modules

fix #84
@JocelynDelalande

This comment has been minimized.

JocelynDelalande commented Mar 1, 2018

Ça me semble bien en l'état, à vous de jouer :)

Une petite capture de la page d'accueil sans le bloc de statistiques (dans le cas où on l'a désactivé dans les préférences d'affichage)

image

@JocelynDelalande JocelynDelalande changed the title from WIP: Tableau de bord / page d'accueil configurable to Tableau de bord / page d'accueil configurable Mar 1, 2018

@jkiwi

This comment has been minimized.

jkiwi commented Mar 2, 2018

pour moi c'est très bien, et ça ouvre la voie à l'ajout éventuel de module futur affichable en page d'accueil...

@jbgury

This comment has been minimized.

Contributor

jbgury commented Mar 2, 2018

ok pour moi

Garthylou Jean-Baptiste Gury added some commits Mar 2, 2018

@jbgury

jbgury approved these changes Mar 2, 2018

@jbgury jbgury merged commit 1f2959b into libreosteo:develop Mar 2, 2018

@JocelynDelalande JocelynDelalande deleted the JocelynDelalande:jd-configurable-dashboard branch Mar 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment