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

Resolves issue #7 #15

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

Resolves issue #7 #15

wants to merge 19 commits into from

Conversation

JulienWilhelm
Copy link
Contributor

Main changes :

No breaking changes.
Should be ready as v1.1.0.

@JulienWilhelm JulienWilhelm changed the title Resolve issue #7 Resolves issue #7 Jan 12, 2023
@JulienWilhelm
Copy link
Contributor Author

@supertanuki ?

@supertanuki
Copy link
Contributor

@JulienWilhelm merci, je regarde quand je peux, je rajoute dans ma todo

@supertanuki
Copy link
Contributor

NumÉcoDiag is now able to be served as webApp for debugging,

Le documenter dans le readme ?

#7
La solution proposée est à affiner. J'ai noté des effets de bord, par exemple lorsque l'onglet Options est affiché, puis on modifie des valeurs conforme/non conforme dans la popup, les données exportées ne contiennent pas ces valeurs.

@JulienWilhelm
Copy link
Contributor Author

Effectivement, j'avais noté cette absence de synchronisation.
Je me pencherai dessus pour voir comment y remédier.

@JulienWilhelm
Copy link
Contributor Author

@supertanuki C'est fait !

@AMorgaut
Copy link

cette PR n'est pas encore mergée
un soucis ?

@JulienWilhelm
Copy link
Contributor Author

@AMorgaut Il me semble que personne n'a repris la gestion de ce projet :/.
Entretemps, le référentiel a par ailleurs évolué.

Pour info, côté Temesis, j'ai repris le projet en interne, sur cette base, et ça marche vraiment bien pour nos audits.
À l'heure actuelle, nous n'avons toutefois pas publié le code.

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.

3 participants