microsoft / vsts-extension-retrospectives

An Azure DevOps extension for efficient retrospectives
MIT License
183 stars 82 forks source link

Obscure Feedback Feature - Blurring Own Feedback + Can't Edit Once Entered #777

Closed tyfeather closed 3 months ago

tyfeather commented 4 months ago

Describe the bug When the 'obscure feedback' setting is enabled, users are unable to edit their feedback once submitted and their submitted feedback is also blurred as if it were another user's.

Are you using the Retrospectives Tool on-prem? [ ] Yes [x] No

If Yes, what version of the extension?

To Reproduce Steps to reproduce the behavior:

  1. Create new board with 'Obscure Feedback' setting enabled.
  2. Create feedback card.
  3. Note feedback is blurred and unable to edit once entered.

Expected behavior Users should not see their own feedback blurred and they should be able to edit/adjust after submitted.

Screenshots If applicable, add screenshots to help explain your problem. If you have any developer console information that feels relevant, that is also helpful!

What screen type are you seeing this on ? [x] Desktop [ ] Mobile

Additional context Add any other context about the problem here.

Retro Bug 2 Retro Bug

JanaVPetrova commented 4 months ago

+1 I also experience the same issue.

WilliamDAssafMSFT commented 4 months ago

Experiencing the same issue. Hard to add new feedback to a blurred textbox in Collect.

image image

This appears on a retro that was created about a month ago in early June, with the image option, as well as newly created retros with that option.

agnieszkapuzio commented 4 months ago

It's also happening on on-prem Azure DevOps

jeremyperez-kimballmidwest commented 3 months ago

+1 same issue. Also, it appears to be linked to the "Do not display names in feedback" option.

rjhennessey commented 3 months ago

My team is seeing this behaviour as well. This week is the first time we noticed it, since our last Retros were two weeks ago. It was fine two weeks ago, as far as I know..

taspeotis commented 3 months ago

I suspect it's related to

https://github.com/microsoft/vsts-extension-retrospectives/pull/772

The frontend decides whether to blur or not thus:

    const hideFeedbackItems = this.props.hideFeedbackItems && (this.props.userIdRef !== getUserIdentity().id);