Added new forceAutoNumber parameter to import_records method.#86
Added new forceAutoNumber parameter to import_records method.#86sburns merged 1 commit intoredcap-tools:masterfrom
Conversation
|
Woah I'm glad they added this to the API! Our test suite is in shambles now, have you run this locally to verify it works? |
|
Yes, I ran multiple local tests and I'm also using the fork with this change in a script we are running in production. So far so good. I wanted to write a test, but I saw the issue with the failing REDCap instance used in tests. |
|
Do you have access to an instance we could use for our tests? |
|
Hi @CarlosBorroto, a separate PR implemented tests that don't need a redcap instance, would you mind back-merging master? We should have green tests then. |
|
@sburns will do. Sorry I missed you previous question. I don't have access to an instance. I'm working with a collaborator which is using redcap. |
|
Hey guys. I run a training instance of REDCap that is public and has table authentication, please email me firas.wehbe [chez] northwestern edu, and I will be happy to provision you as user if still needed for these tests or others needed in future PRs |
|
@firaswehbe #85 moved our tests to not requiring a running instance (and making http requests to a running service in a testsuite is icky anyways) so while I very much appreciate the offer, I don't think we need it now. |
43ea665 to
1b95640
Compare
|
@sburns I finally got around to update this PR. Tests are now passing. Nice job removing the dependency on a running instance. |
We would like to use the
forceAutoNumberparameter of theimport_recordsAPI endpoint. This allows auto-numbering project feature to ignore therecord_idprovided by the user and instead use auto-incremented record ids as per project setting.