community-platform icon indicating copy to clipboard operation
community-platform copied to clipboard

Questions: title of the tab changes on clicking question title

Open ADTmux opened this issue 1 year ago • 3 comments

PR Checklist

PR Type

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] Developer experience (improves developer workflows for contributing to the project)

Description

Clicking a question's title changes the name of the tab to reflect that question.

Git Issues

Closes #3409


ADTmux avatar Apr 13 '24 10:04 ADTmux

Please let me know if a different workaround was required, or a different approach was expected! I'd be happy to work on that as well!

ADTmux avatar Apr 13 '24 10:04 ADTmux

hey @benfurber , tested on Cypress and did the requested change. Please let me know what next is required! :)

ADTmux avatar Apr 15 '24 11:04 ADTmux

@ADTmux Sorry to request another change on a tiny commit(!) but I think it should be set during the useeffect rather than the render. Let's go with just after setQuestion(foundQuestion)

The branch is also failing the linting right now. That's easiest to fix by running yarn format.

benfurber avatar Apr 17 '24 12:04 benfurber

Hi @benfurber could you test it now?

ADTmux avatar Apr 30 '24 09:04 ADTmux

Awesome. Thanks @ADTmux.

So I think that updating the commit messages to our style will fix the linter plus a rebase with master and we're good to go.

benfurber avatar Apr 30 '24 13:04 benfurber

@benfurber Done!

ADTmux avatar May 02 '24 01:05 ADTmux

Hey @ADTmux. Thanks for making that change. Looks like you've pulled in some unrelated (and wrong) changes too? The stuff incrementViewCount on src/pages/Question/QuestionPage.tsx shouldn't be included. I think then the failing unit test will pass.

benfurber avatar May 06 '24 13:05 benfurber

Just checking in @ADTmux. Have you seen my last comment? :)

benfurber avatar May 13 '24 09:05 benfurber

Closing this as I've sorted it in #3623

benfurber avatar Jun 05 '24 16:06 benfurber