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

Unexpected Behavior: redcap_event_instruments does not return all arms by default #482

Closed
januz opened this issue May 14, 2023 · 2 comments
Assignees

Comments

@januz
Copy link
Contributor

januz commented May 14, 2023

Describe the behavior:

I feel like a better default behavior for REDCapR::redcap_event_instruments() would be to return event-form mappings for all arms instead of for arm 1. Additionally, I was expecting that I could specify all arms as a character, e.g., arms = c("1", "2", "3") but this also only returns mappings for arm 1, i.e., one seems to only be able to export mappings for one arm at a time.

Expected behavior:

Default behavior should be like redcapAPI::exportMappings(), i.e., return mappings for all arms. If a user specifies a specific arm or a vector of selected arms, mapping for this / those arm(s) should be returned.

Desktop (please complete the following information):

  • OS: UNIX, Mac
  • REDCapR: Version ‘1.1.9005’
@wibeasley wibeasley self-assigned this May 26, 2023
wibeasley added a commit that referenced this issue May 26, 2023
wibeasley added a commit that referenced this issue May 26, 2023
wibeasley added a commit that referenced this issue May 26, 2023
wibeasley added a commit that referenced this issue May 26, 2023
wibeasley added a commit that referenced this issue May 26, 2023
wibeasley added a commit that referenced this issue May 26, 2023
wibeasley added a commit that referenced this issue May 26, 2023
wibeasley added a commit that referenced this issue May 26, 2023
wibeasley added a commit that referenced this issue May 26, 2023
@wibeasley
Copy link
Member

@januz, does this work like you want now?

@januz
Copy link
Contributor Author

januz commented Jul 26, 2023

@wibeasley Sorry for the late reply and generally not having been in contact about the other issue we were talking about (unfortunately, work has been and continues to be crazy...). I just tested the new version and it works as expected. Thanks so much for implementing this!

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

No branches or pull requests

2 participants