NotificationsYou must be signed in to change notification settings
Fork21
Star8

fix(PM-3090): likes dislikes not updated automatically#1351

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

hentrymartin wants to merge3 commits intodev

base:dev

Choose a base branch

frompm-3090

Open

fix(PM-3090): likes dislikes not updated automatically#1351

hentrymartin wants to merge3 commits intodevfrompm-3090

Conversation

Copy link

Collaborator

hentrymartin commentedNov 30, 2025

Related JIRA Ticket:

https://topcoder.atlassian.net/browse/PM-3090

What's in this PR?

Seems like the swr url got changed slighly and it broke the mutation
Now updated the mutation url keys to reflect the change
This would make the likes/dislikes/comments to be updated automatically.

hentrymartin added2 commits

November 30, 2025 16:32

fix: likes dislikes not updated automatically

ca06ce4

fix: likes dislikes not updated automatically

44c4edf

hentrymartin requested review fromjmgasper andkkartunov ascode owners

November 30, 2025 15:34

github-actionsbot reviewed

Nov 30, 2025

View reviewed changes

...iew/src/lib/components/Scorecard/ScorecardViewer/ScorecardQuestion/AiFeedback/AiFeedback.tsx OutdatedShow resolvedHide resolved

github-actionsbot reviewed

Nov 30, 2025

View reviewed changes

...mponents/Scorecard/ScorecardViewer/ScorecardQuestion/AiFeedbackActions/AiFeedbackActions.tsx


		try{
		constitemsKey=`${EnvironmentConfig.API.V6}/workflows/${workflowId}/runs/${workflowRun.id}/items`
		constitemsKey=`${EnvironmentConfig.API.V6}/workflows/${workflowId}/runs/${workflowRun.id}/items?[${workflowRun?.status}]`

Copy link

github-actionsbotNov 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

[❗❗correctness]
The use of optional chaining withworkflowRun?.status in the URL may lead to unexpected behavior ifworkflowRun isundefined. Consider ensuringworkflowRun is defined before constructing the URL to avoid potential issues.