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: add publish function on list #1473

Merged
merged 5 commits into from
Oct 22, 2024
Merged

feat: add publish function on list #1473

merged 5 commits into from
Oct 22, 2024

Conversation

m-maillot
Copy link
Contributor

No description provided.


const handleSelectAllClick = (event: React.ChangeEvent<HTMLInputElement>) => {
setItemCheck(
rows.reduce((acc, item) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Tu peux passer les typages en générique dans le reduce, je trouve ca plus sympa qu'un as

Suggested change
rows.reduce((acc, item) => {
rows.reduce<ItemCheck>((acc, item) => {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui en effet, bien plus sympa ! :)

Copy link

sonarcloud bot commented Oct 22, 2024

@m-maillot m-maillot enabled auto-merge (squash) October 22, 2024 11:52
Copy link

🎉 Deployment for commit 34a7711 :

Ingresses
Docker images
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/export:sha-34a77110ca10b51c35a65d7d7171e6dc9fa88542
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/frontend:sha-34a77110ca10b51c35a65d7d7171e6dc9fa88542
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/hasura:sha-34a77110ca10b51c35a65d7d7171e6dc9fa88542
Debug

@m-maillot m-maillot merged commit 2a4808d into master Oct 22, 2024
25 checks passed
@m-maillot m-maillot deleted the feat/publish-select branch October 22, 2024 12:01
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.

3 participants