-
Notifications
You must be signed in to change notification settings - Fork 139
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[$30] Copilot Feedback Form #4399
Comments
fyi @vikasrohit |
@yoution can you please wrap this one quickly? |
@vikasrohit please assign to me |
@vikasrohit please review #4402 |
@yoution it looks good. Can you please style the Feedback button to look like |
@vikasrohit |
change to this style, please review #4404 |
@yoution screenshot looks good, I have just merged it to feature branch. Will check on dev and come back to you. I have also updated the prize money for it, let me know if it is not okay. |
* fix: issue #4399 * Temp deployable feature branch * fix: issue #4399 Co-authored-by: yoution <[email protected]>
-- Verified Copilot feedback form for Manager role. |
-- Team member is able to see the feedback form as well - pls clarify regarding this. |
@lakshmiathreya good catch here. We haven't thought of role based access yet. We can log it as separate issue and handle it later.
I think that is in fact the requirement, we want any project member to be able to do so, however, we just need to confirm about copilots. Most probably we won't allow copilots to give feedback for themselves. :) @ajefts what are your thoughts? |
yes @vikasrohit - Team member feedback is the primary intent I guess. Just need clarification regarding form visibility for Copilot. |
Lets log a separate issue for that, we will get it fixed in next cycle. |
The form visibility needs to be Role based. Closing the Issue - pending fix for Copilot role visibility tracked in #4420 |
New Feature:
Add a feature to allow a project user to leave feedback for the copilots on the project.
We have a typeform to handle the capture of the feedback: https://topcoder.typeform.com/to/vgqiBXdk#copilot=xxxxx&projectid=xxxxx&submitter=xxxxx
Let's add a link/action next to each copilot so that a project user can leave individual feedback for each copilot:
Note that the TypeForm URL needs to have the copilot handle, project_id, and handle of the person submitting the feedback.
Nice to have: Do not provide the feedback option to Copilots on the project. If the current user is a copilot on the project, then hide the feedback links.
The text was updated successfully, but these errors were encountered: