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(contributions): ajouter des infographies #1544

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

m-maillot
Copy link
Contributor

No description provided.

# Conflicts:
#	targets/frontend/src/components/forms/EditionField/MenuSpecial.tsx
@tokenbureau tokenbureau bot temporarily deployed to review-linked-feat-infographic-3qupee February 3, 2025 15:37 Destroyed
@tokenbureau tokenbureau bot temporarily deployed to review-linked-feat-infographic-3qupee February 6, 2025 10:07 Destroyed
@m-maillot m-maillot force-pushed the linked/feat/infographic branch from b26aee5 to 3efe729 Compare February 6, 2025 11:15
@tokenbureau tokenbureau bot temporarily deployed to review-linked-feat-infographic-3qupee February 6, 2025 11:16 Destroyed
@m-maillot m-maillot force-pushed the linked/feat/infographic branch from 3efe729 to 07255a1 Compare February 6, 2025 13:29
@tokenbureau tokenbureau bot temporarily deployed to review-linked-feat-infographic-3qupee February 6, 2025 13:29 Destroyed
@m-maillot m-maillot requested review from maxgfr, Viczei and carolineBda and removed request for Viczei February 7, 2025 08:43
# Conflicts:
#	targets/frontend/src/components/forms/EditionField/MenuSpecial.tsx

# Conflicts:
#	targets/frontend/src/components/forms/EditionField/Editor.tsx
#	targets/frontend/src/components/forms/EditionField/extensions/Infographic.ts
@m-maillot m-maillot force-pushed the linked/feat/infographic branch from 07255a1 to b42b05c Compare February 10, 2025 12:59
@m-maillot m-maillot deployed to review-auto February 10, 2025 12:59 — with GitHub Actions Active
Copy link

@tokenbureau
Copy link

tokenbureau bot commented Feb 10, 2025

🎉 Deployment for commit b42b05c :

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

Copy link
Member

@maxgfr maxgfr left a comment

Choose a reason for hiding this comment

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

GG sympa l'extension maison pour tiptap

Comment on lines +28 to +33
ARG NEXT_PUBLIC_BUCKET_PUBLIC_ENDPOINT
ENV NEXT_PUBLIC_BUCKET_PUBLIC_ENDPOINT=$NEXT_PUBLIC_BUCKET_PUBLIC_ENDPOINT
ARG NEXT_PUBLIC_BUCKET_DEFAULT_FOLDER
ENV NEXT_PUBLIC_BUCKET_DEFAULT_FOLDER=$NEXT_PUBLIC_BUCKET_DEFAULT_FOLDER
ARG NEXT_PUBLIC_BUCKET_DRAFT_FOLDER
ENV NEXT_PUBLIC_BUCKET_DRAFT_FOLDER=$NEXT_PUBLIC_BUCKET_DRAFT_FOLDER
Copy link
Member

Choose a reason for hiding this comment

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

yes, ça me semble bon comme ça !

Comment on lines +22 to +30
import {
Button,
DialogActions,
DialogContentText,
TextField,
} from "@mui/material";
import DialogTitle from "@mui/material/DialogTitle";
import Dialog from "@mui/material/Dialog";
import DialogContent from "@mui/material/DialogContent";
Copy link
Member

Choose a reason for hiding this comment

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

ça nous sauve tellement la vie cette lib :p

Comment on lines 43 to +46

type ModeCreation = {
mode: 0;
};
Copy link
Member

Choose a reason for hiding this comment

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

on peut laisser les types ici, car on ne les partage pas, c'est bien ça ?

Comment on lines +105 to +107
addCommands() {
return {
setInfographic:
Copy link
Member

Choose a reason for hiding this comment

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

bon à savoir, si on veut l'améliorer dans le futur

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