openedx / frontend-app-discussions

A React-based micro frontend for the Open edX discussion forums.
GNU Affero General Public License v3.0
6 stars 62 forks source link

Fix bug with a repeated search query #540

Closed DmytroAlipov closed 1 year ago

DmytroAlipov commented 1 year ago

Description

An issue has been found with research if no changes are made to the search field:

This fix allows actions to be ignored if the current search query matches a previous query. If there are any changes in the search query, then it is sent to the backend.

openedx-webhooks commented 1 year ago

Thanks for the pull request, @DmytroAlipov! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

itsjeyd commented 1 year ago

Hey @DmytroAlipov, thank you for this contribution! It looks like the changes are ready for review?

awais-ansari commented 1 year ago

@DmytroAlipov Thank you for this contribution! The Team will review this PR Shortly.

DmytroAlipov commented 1 year ago

@awais-ansari just a friendly ping, all tests passed ;)

DmytroAlipov commented 1 year ago

I updated brunch.

openedx-webhooks commented 1 year ago

@DmytroAlipov 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.