Skip to content

Conversation

@Grafikart
Copy link
Collaborator

@Grafikart Grafikart commented Dec 11, 2025

Add conditions to display the modal for the questionnaire and replace the useArticulationTable

Need InseeFr/Drama-Queen#392 to work

Fix #257
Fix #258

Copy link
Collaborator

@laurentC35 laurentC35 left a comment

Choose a reason for hiding this comment

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

Some review, but i think we need a merge with main before

...surveyUnit,
priority: true,
});
navigate(`/queen/survey-unit/${id}/synchronize`);
Copy link
Collaborator

Choose a reason for hiding this comment

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

It must be /queen/interrogations/${id}/synchronize

But i think, your started branch is outdated with main

const handleOpenModal = () => {
const openQuestionnaire = () => {
if (!isWebQuestionnaire) {
navigate(`/queen/survey-unit/${id}`);
Copy link
Collaborator

Choose a reason for hiding this comment

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

It must be /queen/interrogations/${id}

But i think, your started branch is outdated with main

@EmmanuelDemey EmmanuelDemey force-pushed the feat/other-mode-notification branch from d563104 to 34fd5ad Compare January 2, 2026 10:22
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.

4 participants