chore: Tags and notes to csv exports

[Dhruwang]

What does this PR do?

Fixes # 1785

How should this be tested?

Download CSV

Checklist

Required

• Filled out the "How to test" section in this PR
• Read How we Code at Formbricks
• Self-reviewed my own code
• Commented on my code in hard-to-understand bits
• Ran pnpm build
• Checked for warnings, there are none
• Removed all console.logs
• Merged the latest changes from main onto my branch with git pull origin main
• My changes don't cause any responsiveness issues
• First PR at Formbricks? Please sign the CLA! Without it we wont be able to merge it 🙏

Appreciated

• If a UI change was made: Added a screen recording or screenshots to this PR
• Updated the Formbricks Docs if changes were necessary

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments

Name	Status	Preview	Comments	Updated (UTC)
formbricks-cloud	⬜️ Ignored (Inspect)	Visit Preview		Jan 29, 2024 2:28pm
formbricks-com	⬜️ Ignored (Inspect)	Visit Preview		Jan 29, 2024 2:28pm

[github-actions]

Thank you for following the naming conventions for pull request titles! 🙏

[review-agent-prime]

apps/web/app/(app)/environments/[environmentId]/surveys/[surveyId]/components/CustomFilter.tsx

Consider breaking down the getMatchQandA function into smaller functions, each with a single responsibility. This would make the code easier to read and maintain.
Create Issue
See the diff
Checkout the fix

    const mapQuestionIdToHeadline = (survey: TSurvey) => {
      const questionIdToHeadline = {};
      survey.questions.forEach((question) => {
        questionIdToHeadline[question.id] = question.headline;
      });
      return questionIdToHeadline;
    };

    const updateResponses = (responses: TResponse[], questionIdToHeadline: any) => {
      // ... existing code to update responses ...
    };

    const getMatchQandA = (responses: TResponse[], survey: TSurvey) => {
      if (survey && responses) {
        const questionIdToHeadline = mapQuestionIdToHeadline(survey);
        const updatedResponses = updateResponses(responses, questionIdToHeadline);
        // ... existing code to return updatedResponsesWithTags ...
      }
      return [];
    };

git fetch origin && git checkout -b ReviewBot/The-f-heqxj4o origin/ReviewBot/The-f-heqxj4o

Consider breaking down the downloadResponses function into smaller functions, each with a single responsibility. This would make the code easier to read and maintain.
Create Issue
See the diff
Checkout the fix

    const prepareFileResponse = (response: any, metaDataFields: string[], hiddenFieldIds: string[], questionNames: string[]) => {
      // ... existing code to prepare fileResponse ...
    };

    const downloadResponses = useCallback(
      async (filter: FilterDownload, filetype: "csv" | "xlsx") => {
        // ... existing code to prepare downloadResponse, questionNames, hiddenFieldIds, metaDataFields, userAttributes, matchQandA ...
        const jsonData = matchQandA.map((response) => {
          const fileResponse = prepareFileResponse(response, metaDataFields, hiddenFieldIds, questionNames);
          // ... existing code to return fileResponse ...
        });
        // ... existing code to download file ...
      },
      [downloadFileName, responses, survey, extracMetadataKeys, getAllResponsesInBatches]
    );

git fetch origin && git checkout -b ReviewBot/The-`-my93z5q origin/ReviewBot/The-`-my93z5q

[jobenjada]

hey Dhru! Looks great, thanks :) Now that we're at this, please:

1. Remove the Formbricks User Id from all exports
2. Add userId fill it when the data is available

Thanks!