-
Notifications
You must be signed in to change notification settings - Fork 4
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
#2771 - TECH - éliminer les occurences de pole emploi au profit de france travail (première partie) #2816
Merged
JeromeBu
merged 5 commits into
main
from
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
Jan 20, 2025
The head ref may contain hidden characters: "2771-tech-\u00E9liminer-les-occurences-de-pole-emploi-au-profit-de-france-travail"
Merged
#2771 - TECH - éliminer les occurences de pole emploi au profit de france travail (première partie) #2816
JeromeBu
merged 5 commits into
main
from
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
Jan 20, 2025
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
JeromeBu
force-pushed
the
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
branch
4 times, most recently
from
January 15, 2025 10:34
b0a8eca
to
9c427e9
Compare
JeromeBu
changed the title
2771 tech éliminer les occurences de pole emploi au profit de france travail
#2771 - TECH - éliminer les occurences de pole emploi au profit de france travail (première partie)
Jan 15, 2025
Review app: |
JeromeBu
force-pushed
the
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
branch
from
January 16, 2025 15:05
9c427e9
to
2168a81
Compare
bbohec
requested changes
Jan 16, 2025
...src/domains/agency/adapters/pe-agencies-referential/HttpPeAgenciesReferential.manual.test.ts
Outdated
Show resolved
Hide resolved
back/src/domains/convention/adapters/france-travail-gateway/FrancetTravailRoutes.ts
Outdated
Show resolved
Hide resolved
back/src/domains/convention/adapters/france-travail-gateway/FrancetTravailRoutes.ts
Outdated
Show resolved
Hide resolved
...c/domains/convention/adapters/france-travail-gateway/HttpFranceTravailGateway.manual.test.ts
Show resolved
Hide resolved
...rc/domains/core/authentication/ft-connect/adapters/ft-connect-gateway/ftConnectApi.routes.ts
Show resolved
Hide resolved
bbohec
force-pushed
the
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
branch
from
January 20, 2025 10:29
2168a81
to
975d340
Compare
bbohec
force-pushed
the
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
branch
from
January 20, 2025 11:10
975d340
to
9287ada
Compare
JeromeBu
force-pushed
the
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
branch
from
January 20, 2025 11:58
9287ada
to
813f54a
Compare
enguerranws
reviewed
Jan 20, 2025
...src/domains/agency/adapters/pe-agencies-referential/HttpFtAgenciesReferential.manual.test.ts
Outdated
Show resolved
Hide resolved
back/src/domains/convention/use-cases/GetConvention.unit.test.ts
Outdated
Show resolved
Hide resolved
...c/domains/convention/adapters/france-travail-gateway/HttpFranceTravailGateway.manual.test.ts
Outdated
Show resolved
Hide resolved
...c/domains/convention/adapters/france-travail-gateway/HttpFranceTravailGateway.manual.test.ts
Outdated
Show resolved
Hide resolved
back/src/domains/convention/use-cases/notifications/NotifyNewConventionNeedsReview.unit.test.ts
Outdated
Show resolved
Hide resolved
.../core/authentication/ft-connect/use-cases/LinkFranceTravailAdvisorAndRedirectToConvention.ts
Outdated
Show resolved
Hide resolved
...entication/ft-connect/use-cases/LinkFranceTravailAdvisorAndRedirectToConvention.unit.test.ts
Outdated
Show resolved
Hide resolved
...entication/ft-connect/use-cases/LinkFranceTravailAdvisorAndRedirectToConvention.unit.test.ts
Outdated
Show resolved
Hide resolved
...authentication/ft-connect/use-cases/NotifyFranceTravailUserAdvisorOnConventionFullySigned.ts
Outdated
Show resolved
Hide resolved
...tion/ft-connect/use-cases/NotifyFranceTravailUserAdvisorOnConventionFullySigned.unit.test.ts
Outdated
Show resolved
Hide resolved
JeromeBu
force-pushed
the
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
branch
from
January 20, 2025 14:57
813f54a
to
9cc48d5
Compare
bbohec
approved these changes
Jan 20, 2025
JeromeBu
force-pushed
the
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
branch
from
January 20, 2025 16:23
9cc48d5
to
04efc16
Compare
JeromeBu
deleted the
2771-tech-éliminer-les-occurences-de-pole-emploi-au-profit-de-france-travail
branch
January 20, 2025 16:31
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Dans cette PR je me contente d'enlever les occurences dans le code, sans toucher à ce qui est plus sensible : on touche pas aux URL ni à la DB. Ça sera pour d'autres PR, plus scopées