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

Make modals persistents #639

Merged
merged 26 commits into from
Jul 14, 2024
Merged

Make modals persistents #639

merged 26 commits into from
Jul 14, 2024

Conversation

monsieurswag
Copy link
Contributor

No description provided.

@monsieurswag monsieurswag marked this pull request as draft July 9, 2024 15:48
@monsieurswag monsieurswag force-pushed the make_modals_persistents branch from 2c88317 to 6bc27e6 Compare July 10, 2024 12:30
@monsieurswag monsieurswag force-pushed the make_modals_persistents branch from 3fcbaf2 to 0620a1b Compare July 10, 2024 13:40
@monsieurswag monsieurswag marked this pull request as ready for review July 10, 2024 13:41
@monsieurswag monsieurswag force-pushed the make_modals_persistents branch 3 times, most recently from 8ce351f to 750470e Compare July 11, 2024 15:31
…tion that don't success to redirect the client's location to /my-profile for whatever incomprehensible reason

Remove tag
…have been executed in gitub actions

Correct debug
@monsieurswag monsieurswag force-pushed the make_modals_persistents branch from 70a1fee to 7d9f06c Compare July 12, 2024 10:33
@monsieurswag monsieurswag force-pushed the make_modals_persistents branch from a12adff to b070250 Compare July 12, 2024 11:11
@monsieurswag monsieurswag force-pushed the make_modals_persistents branch from d1fc4b6 to c4d647d Compare July 12, 2024 21:22
@monsieurswag monsieurswag force-pushed the make_modals_persistents branch from 71c5751 to d793a0c Compare July 12, 2024 21:31
@monsieurswag
Copy link
Contributor Author

Functional tests are fixed, we just have to fix the frontend linter issue then we can merge the PR.

@eric-intuitem
Copy link
Collaborator

fixes #623

did npm install --save-dev prettier before
Copy link
Collaborator

@eric-intuitem eric-intuitem left a comment

Choose a reason for hiding this comment

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

LGTM

@ab-smith ab-smith merged commit 43de0c8 into main Jul 14, 2024
12 checks passed
@ab-smith ab-smith deleted the make_modals_persistents branch July 14, 2024 14:57
@github-actions github-actions bot locked and limited conversation to collaborators Jul 14, 2024
@ab-smith
Copy link
Contributor

the cancel does clear out the modal but not the save, we need that as well

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants