Skip to content

Next unresolved discussion button takes user to the right place

Illya Klymov requested to merge xanf-fix-unresolved-discussion-jump into master

What does this MR do?

This MR fixes "next unresolved discussion button" failing to switch tabs when we' re on diffs tab and no unresolved discussions remained on diffs

This is done by making discussion_navigation mixin aware of entire discussion entry instead of just id - so we could check if our discussion is a diff and properly switch tabs if required.

Screenshots

Demo

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Related issue: #35360 (closed)

Edited by 🤖 GitLab Bot 🤖

Merge request reports