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

Version 2.0.0 #17

Merged
merged 11 commits into from
May 24, 2024
Merged

Version 2.0.0 #17

merged 11 commits into from
May 24, 2024

Conversation

ChemiKyle
Copy link
Contributor

@ChemiKyle ChemiKyle commented Mar 18, 2024

Closes #16

Replaces #11 using checkboxes as detailed in #11 (comment)

Refactors module to use framework version 15.

Needs a commit to update the README to reflect breaking changes and the new checkbox fields.

Note that there is currently no automated migration provided from v1.x.x to v2.0.0 as redcap_module_system_change_version is deprecated and redcap_module_system_enable does not give me access to the framework, so I can't even use getProjectsWithModuleEnabled for manual migration let alone any of the nicer functions for detecting config to infer version. Users will unfortunately have to reselect their scoping; CSS and form scoping should be retained though.

@ChemiKyle ChemiKyle requested a review from pbchase March 18, 2024 21:22
@ChemiKyle ChemiKyle changed the title Refactor to Version 2.0.0 Mar 18, 2024
@ChemiKyle
Copy link
Contributor Author

@lsgs, @h3n0r1k, and @colinaut I welcome y'all's input as well!

@pbchase
Copy link
Contributor

pbchase commented Apr 23, 2024

@saipavan10-git, could you document the breaking change and write some SQL code to help people get form 1.x to 2.0.0?

  • Revise the README to note that 2.0.0 is a breaking change.
  • Suggest that REDCap admins run a block of SQL against the REDcap backend
  • Write that block of SQL

Consult with me if you have trouble.

@pbchase
Copy link
Contributor

pbchase commented May 23, 2024

@saipavan10-git , the old screenshot is now out of date. Could you update it with a v2.0.0 screenshot and double-check that the text still makes sense?

Updated the README to have only the image of new configuration. Also removed the old config image from the image assets.
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.

None yet

3 participants