-
Notifications
You must be signed in to change notification settings - Fork 490
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
Modified the external results sample format #6694
Modified the external results sample format #6694
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You'll have to update the configs here too: https://github.com/coronasafe/care_deploy_configs
cc: @sainak
handleDownload function in ExternalResultUpload.tsx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@konavivekramakrishna just some minor alterations
WHAT
🤖[deprecated] Generated by Copilot at ed80a98
The pull request adds a feature to download the external result template file locally and updates the template file with a sample row of data. It modifies the
src/Components/ExternalResult/ExternalResultUpload.tsx
,public/config.json
, andpublic/External-Results-Template.csv
files.Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖[deprecated] Generated by Copilot at ed80a98
External-Results-Template.csv
(link)config.json
to store a relative path to the template file instead of a Google Sheets link (link)handleDownload
in the external result upload pageExternalResultUpload.tsx
that uses the config value to download the template file locally (link)handleDownload
function (link)