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(iCherche): add a config to hide advanced options #1386

Merged
merged 5 commits into from
Sep 18, 2023

Conversation

alecarn
Copy link
Collaborator

@alecarn alecarn commented Sep 15, 2023

  • feat(iCheche): add a config to hide advanced options
  • fix(iCherche): edit translation

@alecarn alecarn self-assigned this Sep 15, 2023
@alecarn alecarn requested a review from pelord September 15, 2023 17:39
@alecarn alecarn changed the title 1011-icherche-simplifier-le-menu-de-configuration-1 feat(iCheche): add a config to hide advanced options Sep 15, 2023
@alecarn alecarn added this to In Review in IGO2 - Roadmap V16.0.0 via automation Sep 15, 2023
@alecarn alecarn changed the title feat(iCheche): add a config to hide advanced options feat(iCherche): add a config to hide advanced options Sep 15, 2023
ecmax?: string;
page?: string;
type?: string;
showAdvanced?: boolean;
Copy link
Member

Choose a reason for hiding this comment

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

vraiment un param passé à iCherche? Est-ce coté client seulement? Si c'Est le cas, je crois que ca n'a pas sa place la ;(.

Copy link
Member

@pelord pelord left a comment

Choose a reason for hiding this comment

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

Si le coeur te le dit, bonifier la doc sur l'assemblage.
image

IGO2 - Roadmap V16.0.0 automation moved this from In Review to QA Sep 18, 2023
@pelord pelord merged commit 0fee36e into next Sep 18, 2023
4 checks passed
IGO2 - Roadmap V16.0.0 automation moved this from QA to Done Sep 18, 2023
@pelord pelord deleted the 1011-icherche-simplifier-le-menu-de-configuration-1 branch September 18, 2023 15:17
@alecarn alecarn removed this from Done in IGO2 - Roadmap V16.0.0 Sep 18, 2023
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.

None yet

2 participants