-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat(bouquet): draft for admins #344
Conversation
✅ Deploy Preview for ecospheres ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for meteo-france ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Related ecolabdata/ecospheres#88 |
Top ! Est-ce qu'on ajouterait pas le même mécanisme pour un user sur ses propres bouquet ? Si c'est pas incompatible avec une approche par orga dans un deuxième temps. J'ai aussi un petit bug de refresh. En tant qu'utilisateur lambda :
=> Le bouquet en cours d'édition s'affiche (sans label "brouillon") jusqu'a ce que je refresh la page. |
Je ne reproduis pas 🤔 Tu vas jusqu'où sur l'édition ? Le bouquet est bien envoyé dans le store avec
Pas incompatible mais potentiellement perte de temps, ça dépend vraiment de comment on gère les droits. Anyway, rien n'empêche de le faire dans une autre PR, même très rapprochée. |
Je vais jusqu'à la page de sélection des jeux de données, sans pour autant en ajouter
Ok, donc plus tard |
Effectivement je reproduis en allant jusque là. Mystère 👀 🔮 🪄.
J'ai travaillé là-dessus dans #345 parce qu'on avait déjà une permission d'édition avec le propriétaire. Je pourrai reprendre la logique ici plus tard. |
Impossible de reproduire ajd... 👻 |
614b14e
to
d703440
Compare
Première solution simple pour #179, en attendant les décisions sur la gestion des droits.
Un refactoring bienvenu de la dépendance circulaire entre UserStore et UserAPI 3e7b406 était nécessaire. Aussi refactoring du composant Tile au passage.
Fix #179