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

fix: only rely on DGEFP for qualiopi certification #1082

Merged
merged 4 commits into from
May 29, 2024
Merged

Conversation

XavierJp
Copy link
Contributor

In addition to more frequent dataset update on recherche-entreprise side, this PR made a change in Organisme de Formation section.

We now only rely on DGEFP response to determine if an organisme is Qualiopi as recherche entreprise can be outdated.

This mean we loose the ability to have "certifié qualiopi" in the section title, but this seem like a fair trade off.

We will still have outdated data in summary and in the search engine so this will not entirely solve the issue, but it will improve it.

Copy link
Contributor

@HAEKADI HAEKADI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Btw after the daily data update, the data will have at worst a 24 hours lag.

@XavierJp
Copy link
Contributor Author

Btw after the daily data update, the data will have at worst a 24 hours lag.

And this is much better, congrats !

@XavierJp XavierJp merged commit 0eb8447 into main May 29, 2024
8 checks passed
@XavierJp XavierJp deleted the FIX-qualiopi branch May 29, 2024 13:33
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.

2 participants