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: Comments link are no longer considered to be displayed - EXO-76566 - Meeds-io/meeds#2771. #4361

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

akhanfir
Copy link
Contributor

Before this change, when create a postX and comment it then get the comment link and open it in new tab, post is displayed. To resolve this problem, add a condition to the creation of the URL to check if the id in the parameter passed in the URL has a text comment if yes opens the comment drawer. After this change, the link is display post and having its comment drawer opened focusing on comment retrieved.

… - Meeds-io/meeds#2771.

Before this change, when create a postX and comment it then get the comment link and open it in new tab, post is displayed. To resolve this problem, add a condition to the creation of the URL to check if the id in the parameter passed in the URL has a text comment if yes opens the comment drawer. After this change, the link is display post and having its comment drawer opened focusing on comment retrieved.
@akhanfir akhanfir requested a review from mkrout January 21, 2025 08:37
@akhanfir akhanfir enabled auto-merge January 21, 2025 08:37
@github-actions github-actions bot added the partialCIBuild Perform Partial CI Build label Jan 21, 2025
@akhanfir akhanfir merged commit fc69c66 into feature/maintenance Jan 21, 2025
7 checks passed
@akhanfir akhanfir deleted the TASK-76566 branch January 21, 2025 12:19
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.

2 participants