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

redcap_write: would like to decide weather writting is 'overwrite' or 'normal' #315

Closed
Auricap opened this issue Nov 20, 2020 · 5 comments
Closed
Assignees

Comments

@Auricap
Copy link

Auricap commented Nov 20, 2020

Hi,
This parameter is set by default to 'overwrite' but I would like that the parameter is set to normal in some cases in which I don't want empty values to overwrite exiting ones. It would be great if this parameter could be set up as input when calling redcap_write.
Thank you!
PS- Great package by the way!very easy to use and user friendly!

@wibeasley wibeasley self-assigned this Nov 20, 2020
wibeasley added a commit that referenced this issue Nov 20, 2020
@wibeasley
Copy link
Member

Hi @Auricap, good suggestion. We haven't had the need for that yet, but happy to add it if it accommodates others. Will you try the dev branch and tell me if it works as expected? If not, please provide a reproducible example.

remotes::install_github(repo="OuhscBbmc/REDCapR", ref="dev")

@wibeasley
Copy link
Member

wibeasley commented Nov 20, 2020

@dmongin, I just noticed that you recently had forked and did something similar --with a little different parameterization. Do you want to see if the new feature meets your needs too?

dmongin@ddd26e7

image

@Auricap
Copy link
Author

Auricap commented Nov 24, 2020

Hi @Auricap, good suggestion. We haven't had the need for that yet, but happy to add it if it accommodates others. Will you try the dev branch and tell me if it works as expected? If not, please provide a reproducible example.

remotes::install_github(repo="OuhscBbmc/REDCapR", ref="dev")

Wow that was fast, thank you! I will take a look to the changes and let you know!

@Auricap
Copy link
Author

Auricap commented Nov 25, 2020

Just tested, works wonders. Looking forward to the prod version. Thanks again!

@dmongin
Copy link

dmongin commented Nov 25, 2020

@dmongin, I just noticed that you recently had forked and did something similar --with a little different parameterization. Do you want to see if the new feature meets your needs too?

dmongin@ddd26e7

image

It is actually my needs, we are working on the same project with Auricap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants