Skip to content

chore: migrate duplicate database implementation in package#2077

Open
shikanime wants to merge 6 commits intomainfrom
pr2077
Open

chore: migrate duplicate database implementation in package#2077
shikanime wants to merge 6 commits intomainfrom
pr2077

Conversation

@shikanime
Copy link
Copy Markdown
Member

@shikanime shikanime commented Apr 10, 2026 •
edited
Loading

Signed-off-by: William Phetsinorath william.phetsinorath-open@interieur.gouv.fr

@shikanime
Copy link
Copy Markdown
Member Author

shikanime commented Apr 10, 2026

#2064

@shikanime shikanime marked this pull request as ready for review April 10, 2026 14:39
@github-actions github-actions Bot added the built label Apr 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 10, 2026 •
edited
Loading

🤖 Hey !

The security scan report for the current pull request is available here.

@shikanime shikanime marked this pull request as draft April 10, 2026 14:44
@shikanime shikanime force-pushed the pr2077 branch 2 times, most recently from 082182c to 8dcfb79 Compare April 13, 2026 09:28
@shikanime shikanime changed the base branch from main to pr2080 April 13, 2026 09:28
@shikanime shikanime mentioned this pull request Apr 13, 2026
Merged
Base automatically changed from pr2080 to main April 13, 2026 13:48
@shikanime shikanime changed the base branch from main to pr2080 April 13, 2026 13:51
@shikanime shikanime marked this pull request as ready for review April 13, 2026 14:04
@shikanime shikanime self-assigned this Apr 15, 2026
@shikanime shikanime added the technical debt Résoud de la dette technique label Apr 15, 2026
@shikanime shikanime changed the base branch from pr2080 to main April 20, 2026 16:17
@shikanime shikanime force-pushed the pr2077 branch 2 times, most recently from 23efa3d to aca7776 Compare April 20, 2026 16:36
@shikanime shikanime marked this pull request as draft April 21, 2026 07:22
@shikanime shikanime marked this pull request as ready for review April 21, 2026 15:34
@shikanime shikanime enabled auto-merge April 21, 2026 15:35
@shikanime shikanime force-pushed the pr2077 branch 4 times, most recently from 60766c1 to 38bed01 Compare April 21, 2026 16:00
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 27, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

3 similar comments
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 27, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 27, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 27, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 27, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 27, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

@shikanime shikanime enabled auto-merge April 27, 2026 16:23
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2026

🤖 Hey !

A preview of the application is available at : https://console-pr-2077.dso.cpin-hp.numerique-interieur.fr

Please be patient, deployment may take a few minutes.

StephaneTrebel
StephaneTrebel previously approved these changes Apr 30, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@StephaneTrebel StephaneTrebel left a comment

Choose a reason for hiding this comment

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

Merci 👍

Comment thread playwright/package.json
@StephaneTrebel StephaneTrebel mentioned this pull request Apr 30, 2026
Merged
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 30, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 30, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

StephaneTrebel
StephaneTrebel previously approved these changes Apr 30, 2026
View reviewed changes
StephaneTrebel
StephaneTrebel previously approved these changes May 4, 2026
View reviewed changes
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

shikanime and others added 6 commits May 4, 2026 15:02
@shikanime @StephaneTrebel
chore: migrate duplicate database implementation in package
a7f392e 
Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
@StephaneTrebel
chore(playwright): limit time wasted in merge queues by failing fast
4d22e43
@StephaneTrebel
fix(database): fix pnpm not being available for PROD images
33c733b 
This means that our prestart npm script cannot run as-is.
I changed it to directly call the prisma script from its node_modules.
@StephaneTrebel
chore: ensure dependencies are correctly linked for server builds
e236f1a 
PNPM has this nasty habit of needing a `pnpm install` to restore links
for internal dependencies. This is a quickfix to ensure shared, etc.
are always built when building server and server-nest.
@StephaneTrebel
chore: add CI related pnpm docker script for troubleshooting
95da9cd
@StephaneTrebel
chore: remove database deploy from dockerfile
2a45dc1
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

@cloud-pi-native-sonarqube
Copy link
Copy Markdown

cloud-pi-native-sonarqube Bot commented May 4, 2026

Passed Quality Gate passed

Issues

Measures

Project ID: cloud-pi-native_console_AYsa46O31ebDtzs-pYyZ

View in SonarQube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@StephaneTrebel StephaneTrebel Awaiting requested review from StephaneTrebel

Assignees

@shikanime shikanime

Labels

built preview Deploy preview app with Argo-cd technical debt Résoud de la dette technique

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@shikanime @StephaneTrebel