Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

fix(admin): allow empty website #254

Merged
merged 2 commits into from
Oct 4, 2022
Merged

Conversation

rap2hpoutre
Copy link
Member

@rap2hpoutre rap2hpoutre commented Sep 29, 2022

Autoriser le fait de rendre le site web vide via l'admin (pour "supprimer" un site web)

@carolineBda Ah, d'ailleurs j'ai fait ce fix en TDD, ça tombe en même temps que ton message (enfin je crois que c'est ça : j'ai codé le test, le test marchait pas, j'ai codé le fix)

@github-actions github-actions bot temporarily deployed to mon-psy-sante-allow-empty-website-admin September 29, 2022 11:02 Inactive
@sonarcloud
Copy link

sonarcloud bot commented Sep 29, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@carolineBda
Copy link
Contributor

Autoriser le fait de rendre le site web vide via l'admin (pour "supprimer" un site web)

@carolineBda Ah, d'ailleurs j'ai fait ce fix en TDD, ça tombe en même temps que ton message (enfin je crois que c'est ça : j'ai codé le test, le test marchait pas, j'ai codé le fix)

Yes top ! Well done

@rap2hpoutre rap2hpoutre merged commit 86a297e into main Oct 4, 2022
@rap2hpoutre rap2hpoutre deleted the allow-empty-website-admin branch October 4, 2022 07:24
SocialGroovyBot added a commit that referenced this pull request Oct 4, 2022
## [1.56.3](v1.56.2...v1.56.3) (2022-10-04)

### Bug Fixes

* **admin:** allow empty website ([#254](#254)) ([86a297e](86a297e))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants