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

Adapt fetch domain status to manage action required cases #686

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

sdemagny
Copy link
Contributor

@sdemagny sdemagny commented Feb 3, 2025

  • corriger les tests du coté de l'admin
  • factoriser les méthodes pour récuperer les checks values

@sdemagny sdemagny force-pushed the sdem/fetch_domain_action_required branch 11 times, most recently from e63de14 to 356bafc Compare February 4, 2025 12:35
@sdemagny sdemagny marked this pull request as ready for review February 4, 2025 12:35
@sdemagny sdemagny requested a review from qbey February 4, 2025 12:36
@sdemagny sdemagny force-pushed the sdem/fetch_domain_action_required branch 2 times, most recently from e240c30 to ab8f5eb Compare February 4, 2025 12:50
Adapt fetch domain status call to manage internal and external
fixes required. Use the new status 'action required' to
manage actions expected from support.
Call a new dimail endpoint to run a fix for internal checks
when all external checks are OK.
@sdemagny sdemagny force-pushed the sdem/fetch_domain_action_required branch from ab8f5eb to 4df4172 Compare February 4, 2025 13:13
@sdemagny sdemagny merged commit 4df4172 into main Feb 4, 2025
19 checks passed
@sdemagny sdemagny deleted the sdem/fetch_domain_action_required branch February 4, 2025 13:24
@sdemagny sdemagny linked an issue Feb 7, 2025 that may be closed by this pull request
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.

Check dimail passage au status action requise
2 participants