website icon indicating copy to clipboard operation
website copied to clipboard

docs: Change feedback form to actionable questions

Open harshsbhat opened this issue 1 year ago • 13 comments

What kind of change does this PR introduce?

Improves the feedback form by adding actionable responses with options YES and NO rather than asking for a descriptive feedback.

Issue Number:

  • Closes #805

Screenshots/videos:

image

Summary

harshsbhat avatar Aug 02 '24 16:08 harshsbhat

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
website ✅ Ready (View Log) Visit Preview c060079867e2fce0771ac77ae79a0b21e5de8c42

github-actions[bot] avatar Aug 02 '24 16:08 github-actions[bot]

Hi @harshsbhat . The CI/CD workflow failed here: https://github.com/json-schema-org/website/actions/runs/10218752435/job/28275614951?pr=841 Can you please check it?

DarhkVoyd avatar Aug 02 '24 17:08 DarhkVoyd

Also, the linked issue is tagged GSoD, so most probably it's part of an existing project board plan/epic. Please ensure that the work to be done is agreed upon and you are assigned the issue/ you have checked with the web team to work on an issue before creating a PR, as mentioned here

DarhkVoyd avatar Aug 02 '24 18:08 DarhkVoyd

I am not very sure what is causing that error. Can you help me? I don't think it is from the file that I have edited

harshsbhat avatar Aug 02 '24 18:08 harshsbhat

Also, the linked issue is tagged GSoD, so most probably it's part of an existing project board plan/epic. Please ensure that the work to be done is agreed upon and you are assigned the issue/ you have checked with the web team to work on an issue before creating a PR, as mentioned here

Okay my bad. I won't start to work on something again without the APPROVAL/ASSIGNMENT for the same. So you want me to keep this PR or archive it?

harshsbhat avatar Aug 02 '24 18:08 harshsbhat

Also, the linked issue is tagged GSoD, so most probably it's part of an existing project board plan/epic. Please ensure that the work to be done is agreed upon and you are assigned the issue/ you have checked with the web team to work on an issue before creating a PR, as mentioned here

Okay my bad. I won't start to work on something again without the APPROVAL/ASSIGNMENT for the same. So you want me to keep this PR or archive it?

@DarhkVoyd

harshsbhat avatar Aug 02 '24 18:08 harshsbhat

Also, the linked issue is tagged GSoD, so most probably it's part of an existing project board plan/epic. Please ensure that the work to be done is agreed upon and you are assigned the issue/ you have checked with the web team to work on an issue before creating a PR, as mentioned here

Okay my bad. I won't start to work on something again without the APPROVAL/ASSIGNMENT for the same. So you want me to keep this PR or archive it?

No worries, Harsh. Let's keep this PR for now. Someone from the web team will review it and provide guidance on how to proceed.

DarhkVoyd avatar Aug 02 '24 18:08 DarhkVoyd

I am not very sure what is causing that error. Can you help me? I don't think it is from the file that I have edited

Basically, the Codecov coverage token is not set up correctly, which is causing the job to fail. You don't need to do anything; the error is in the workflow configuration itself.

aialok avatar Aug 02 '24 18:08 aialok

I am not very sure what is causing that error. Can you help me? I don't think it is from the file that I have edited

Basically, the Codecov coverage token is not set up correctly, which is causing the job to fail. You don't need to do anything; the error is in the workflow configuration itself.

Thanks a lot Alok for helping me out!

harshsbhat avatar Aug 02 '24 18:08 harshsbhat

Hi everyone. After losing the Postman accounts we have lost access to the airtable used to store the data. Moving forward we decided to use google sheets instead and we still need to implement the change. Can we hold this change until we are able to store the data again?

benjagm avatar Aug 07 '24 16:08 benjagm

@harshsbhat still available to work on this issue? We finally fixed the current feedback component to store data again so we can resume the work here and continue with the changes suggested on #762 . Happy to have a chat and work together on this.

benjagm avatar Sep 23 '24 09:09 benjagm

@benjagm yeah sure. I will take a look at this tomorrow and ask some doubts if any. 😊

harshsbhat avatar Sep 26 '24 17:09 harshsbhat

Hi, @harshsbhat it seems like tests are failing and some merge conflicts are there can you pls. fix them

DhairyaMajmudar avatar Oct 03 '24 05:10 DhairyaMajmudar

Thanks for this contribution @harshsbhat . We are going to close this PR as there was no activity in the last 2 months.

benjagm avatar Dec 13 '24 06:12 benjagm