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

feat: passage de la période de publication des offres à 2 mois #717

Merged
merged 8 commits into from
Oct 20, 2023

Conversation

remy-auricoste
Copy link
Collaborator

@remy-auricoste remy-auricoste commented Oct 19, 2023

  • Passage de la période de publication d'une offre de 1 mois à 2 mois
  • suppression de la dépendance directe à moment (la dépendance reste via transitivité des dependances)
  • mutualisation de la constante de période de publication dans une fonction
  • suppression d'une dépendance du frontend à cette période (il en reste 1)

@remy-auricoste remy-auricoste self-assigned this Oct 19, 2023
ui/utils/api.ts Show resolved Hide resolved
@guilletmarion
Copy link
Contributor

Impossible de déposer une offre pour tester : https://www.loom.com/share/74be53896c9f4614952be9f58a6998c1?sid=efa4ba07-3175-4ba9-a862-d92e03f24067

(pb remonté dans la PR 695 également)

@guilletmarion
Copy link
Contributor

Sur la preview https://712.labonnealternance-preview.apprentissage.beta.gouv.fr/espace-pro/creation/entreprise, cela fonctionne en revanche si besoin

@Waltenberg03
Copy link

Impossible de tester en preview le dépôt d’offre au moment de cliquer sur déposer une offre sur la home LBA, rien ne se passe

@remy-auricoste remy-auricoste force-pushed the lbac-1642-periode-expiration-2-mois branch from d53c119 to ae70dff Compare October 20, 2023 09:23
Copy link
Contributor

@guilletmarion guilletmarion left a comment

Choose a reason for hiding this comment

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

Recette ok pour le délai d'expiration à défaut sur 2 mois.
Cas testés : création d'une offre associée à un nouveau compte entreprise, à un compte entreprise existant, à un nouveau compte CFA (offre mandatée), à un compte CFA existant

@github-actions
Copy link

@remy-auricoste remy-auricoste added this pull request to the merge queue Oct 20, 2023
Merged via the queue into main with commit 7571b78 Oct 20, 2023
@remy-auricoste remy-auricoste deleted the lbac-1642-periode-expiration-2-mois branch October 20, 2023 14:57
@github-actions
Copy link

🎉 This PR is included in version 1.135.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants