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

fix: Bad display after deleting a task - EXO-46811 -Meeds-io/meeds#2497. #435

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

akhanfir
Copy link
Contributor

@akhanfir akhanfir commented Oct 15, 2024

Before this change, when creates a task in Task's project and deletes it, the deleted task drawer is still displayed. To resolve this problem, add the closure of this drawer after confirmation of deletion. After this change, The drawer is automatically closed.

Before this change, when creates a task in Task's project and deletes it, the deleted task drawer is still displayed. After this change, The drawer is automatically closed.
@akhanfir akhanfir requested a review from mkrout October 15, 2024 14:25
@github-actions github-actions bot added the partialCIBuild Perform Partial CI Build label Oct 15, 2024
Copy link

sonarcloud bot commented Oct 15, 2024

@rdenarie
Copy link
Member

Hi
@akhanfir Can you update your commit message ?

Your message should answer at least 2 question ?
WHY this change is needed
and
HOW you fixed it ?

In your message, you explain why this change is needed : "when creates a task in Task's project and deletes it, the deleted task drawer is still displayed" which is the answer to the first question, but you dont answer to the HOW.

Please recheck the develop guide https://developer.exoplatform.org/#id-contributions and specially the 1.4.4 section.

Then, can you reword your commit message to explain the HOW

Thanks

@akhanfir akhanfir merged commit dfcfe8f into feature/maintenance Oct 15, 2024
6 checks passed
@akhanfir akhanfir deleted the TASK-46811 branch October 15, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
partialCIBuild Perform Partial CI Build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants