Skip to content
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

Download Template #84 #184

Merged
merged 6 commits into from
Dec 13, 2024
Merged

Download Template #84 #184

merged 6 commits into from
Dec 13, 2024

Conversation

JamesBisese
Copy link
Collaborator

@JamesBisese JamesBisese commented Dec 4, 2024

Added UI element and server code to download a TADA template in excel format.
This is linked to

Added UI element and server code to download a TADA template in excel format
@cristinamullin cristinamullin linked an issue Dec 5, 2024 that may be closed by this pull request
@cristinamullin
Copy link
Collaborator

cristinamullin commented Dec 5, 2024

@JamesBisese I reviewed and made some changes to the upload directions. I also tried to download the template, filled in example data (attached), and uploaded it. I ran into the error you mentioned on a previous call.

Error: (converted from warning) Error in dplyr::group_by: Must group by variables found in `.data`.Column `TADA.CharacteristicName` is not found.

tada_template_example.xlsx

I see that this will be addressed in a separate issue/PR: #135

As mentioned there, when this example file is first brought in, these two TADA functions should be run right away to prepare it for use throughout the rest of the application: TADA_CheckRequiredFields() and then TADA_AutoClean(). TADA_AutoClean() is what creates the TADA.CharacteristicName field which is why we are getting this error.. TADA_AutoClean should be run when a user hits upload.

I started working on this here: fix-option-C-#135

@cristinamullin cristinamullin merged commit 7657570 into develop Dec 13, 2024
6 checks passed
@cristinamullin cristinamullin deleted the download-template-84 branch December 13, 2024 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Option 3 to include "download blank TADA template" button
2 participants