-
Notifications
You must be signed in to change notification settings - Fork 15
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: add data subvention source #1263
Conversation
e2a0f24
to
8aa9f72
Compare
J'aime beaucoup ! Quelques feedbacks UX :
|
slug: data-subvention | ||
short: Data.Subvention | ||
site: https://datasubvention.beta.gouv.fr/ | ||
long: Data.Subvention | ||
logoType: portrait | ||
dataSources: | ||
- label: Data.Subvention | ||
apiSlug: data-subvention | ||
data: | ||
- label: Data.Subvention | ||
targets: | ||
- agent | ||
contact: https://datasubvention.beta.gouv.fr/contact/ | ||
description: Un outil de consultation pour les agents de l'Etat porté par la DJEPVA et la DINUM pour éclaircir vos décisions et mieux connaître les associations |
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.
I am wondering wether we create an administration or we use the existing DJEPVA one
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.
Let's see with the Data.Subvention team ?
if (!subventions) return null; | ||
|
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.
Not sure about this. Either the user has the rights to see it and he/she prefer an explanation like : Aucune demande de subvention n’a été trouvée pour cette association.
. Or h/she is not agent connected and we could display the <AgentWall />
component
95c725f
to
a06ccf8
Compare
Il faudra penser a provisionner DATA_SUBVENTION_API_KEY @MKCG |
Todo
|
Signed-off-by: Xavier Jp <[email protected]>
@rmonnier9 attendons demain pour voir, vu l'incident de prod actuel. |
multiple
.references #1241