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: page info validation #1040

Conversation

Viczei
Copy link
Contributor

@Viczei Viczei commented Sep 28, 2023

No description provided.

@Viczei Viczei self-assigned this Sep 28, 2023
@gitguardian
Copy link

gitguardian bot commented Sep 28, 2023

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@Viczei Viczei changed the base branch from master to 1007-admin-liste-des-contenus-modifiés-lors-dune-maj-des-données September 28, 2023 10:38
@@ -88,7 +89,7 @@
"urql": "^2.0.5",
"uuid": "^8.3.2",
"wonka": "^4.0.15",
"zod": "^3.22.2"
"zod": "3.21.4"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

j'ai baissé la version car la 3.22 n'est pas compatible avec react-hook-form

Comment on lines -20 to -22
if (value === undefined) {
return <></>;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bug d'affichage de la checkbox

@Viczei Viczei merged commit 8d89913 into 1007-admin-liste-des-contenus-modifiés-lors-dune-maj-des-données Sep 28, 2023
4 of 6 checks passed
@Viczei Viczei deleted the feat/page-info-validation branch September 28, 2023 10:42
@sonarcloud
Copy link

sonarcloud bot commented Sep 28, 2023

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

Viczei added a commit that referenced this pull request Oct 6, 2023
… - page info (#1013)

* feat: implementation du schéma de donnée page info

* feat: implementation des pages info

* fix: review

* refactor: implement page info refactor

* feat: implementation page creation

* feat: add upsert + publish

* feat: publish

* feat: publish + refactor to module

* chore: remove npmrc

* fix: publish dep

* feat: page info validation (#1040)

* feat: zod validation page info

* feat: implementation page info validation avec zod

* feat: clean deepPartial

* chore: clean

---------

Co-authored-by: Victor Zeinstra <[email protected]>

* chore: clean

* chore: clean

* chore: use helper text for error

* feat: change url to regex check for files

* chore: add test

* chore: review

* chore: yarn

* refactor: use confirm modal

* chore: downgrade next

* fix: breadcrumb display

* chore: clean

---------

Co-authored-by: Victor Zeinstra <[email protected]>
m-maillot added a commit that referenced this pull request Oct 9, 2023
* feat: implementation du schéma de donnée page info

* feat: implementation des pages info

* fix: review

* refactor: implement page info refactor

* feat: implementation page creation

* feat: add upsert + publish

* feat: publish

* feat: publish + refactor to module

* chore: remove npmrc

* fix: publish dep

* feat: page info validation (#1040)

* feat: zod validation page info

* feat: implementation page info validation avec zod

* feat: clean deepPartial

* chore: clean

---------

Co-authored-by: Victor Zeinstra <[email protected]>

* chore: clean

* chore: clean

* chore: use helper text for error

* feat: change url to regex check for files

* chore: add test

* chore: review

* chore: yarn

* refactor: use confirm modal

* chore: downgrade next

* feat: ajout des validations au formulaire contribution

* feat: gérer le cas des fiches SP

* chore: clean

* feat: contrib answer CC hover + CC search + delete question

* Revert "feat: contrib answer CC hover + CC search + delete question"

This reverts commit 01e1ded.

* chore: add TU

* chore: add TU

* chore: review

* chore: review

* chore: review

* chore: review

* feat: contribution cc hover cc search delete question (#1057)

* feat: contrib answer CC hover + CC search + delete question

* fix: permettre accordéon dans accordéon

* chore: review

* chore: fix test

---------

Co-authored-by: Victor Zeinstra <[email protected]>

* fix: url can be empty

---------

Co-authored-by: Victor Zeinstra <[email protected]>
Co-authored-by: Martial Maillot <[email protected]>
@maxgfr maxgfr removed their request for review October 11, 2023 14:28
m-maillot added a commit that referenced this pull request Oct 24, 2023
… prod et pre-prod (#1064)

* feat: implementation du schéma de donnée page info

* feat: implementation des pages info

* fix: review

* refactor: implement page info refactor

* feat: implementation page creation

* feat: add upsert + publish

* feat: publish

* feat: publish + refactor to module

* chore: remove npmrc

* fix: publish dep

* feat: page info validation (#1040)

* feat: zod validation page info

* feat: implementation page info validation avec zod

* feat: clean deepPartial

* chore: clean

---------

Co-authored-by: Victor Zeinstra <[email protected]>

* chore: clean

* chore: clean

* chore: use helper text for error

* feat: change url to regex check for files

* chore: add test

* chore: review

* chore: yarn

* refactor: use confirm modal

* chore: downgrade next

* feat(MAJ prod): lister les documents déployés lors des mises à jours de prod et pre-prod

* clean-up

* feat(admin) ajout de la page info

* show list docs in table

* fix: doublons sur publier + lien contenus

* chore: fix ts

* fix: remove column cdtnId from queries

* fix: block type not displaying

* fix: graphic validation + order duplicate key

* chore: fix types

* fix: use mapping to create graphql request

* revert

* fix types

* fix: issues with mapping and requests

* fix: issues with publish

* fix: migration order

* chore: clean

* use query instead of API to load docs

* remove unused

* markup

* fix layout

---------

Co-authored-by: Victor Zeinstra <[email protected]>
Co-authored-by: Victor Zeinstra <[email protected]>
Co-authored-by: Martial Maillot <[email protected]>
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.

2 participants