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

mc-9808 Allow a simplified creation for ReferenceTypes and ModelDataTypes #284

Merged
merged 2 commits into from
Mar 5, 2022

Conversation

olliefreeman
Copy link
Contributor

  • Allow POST to DataTypes without a label for RTs and MDTs
  • Generate a default label of "Reference to X" where X is the label of the reference
  • Confirm the existence of the reference
  • If a DT already exists with that default label then generate a new one with the next number in order suffixed
  • Allow POST to DataElements without a label in the DataType for creating new DTs alongside the DE
  • Use the default label for RTs and MDTs when creating a DE if no label supplied
  • Try and find existing DT with same domaintype and label when creating a DE and DT at the same time.
    If the DT exists then use that one rather than creating a new one which will fail as the DT already exists

…ypes

* Allow POST to DataTypes without a label for RTs and MDTs
* Generate a default label of "Reference to X" where X is the label of the reference
* Confirm the existence of the reference
* If a DT already exists with that default label then generate a new one with the next number in order suffixed
* Allow POST to DataElements without a label in the DataType for creating new DTs alongside the DE
* Use the default label for RTs and MDTs when creating a DE if no label supplied
* Try and find existing DT with same domaintype and label when creating a DE and DT at the same time.
 If the DT exists then use that one rather than creating a new one which will fail as the DT already exists
@olliefreeman
Copy link
Contributor Author

test failures are to do with timings, not the code. need to increase the timeout for lock.

@olliefreeman olliefreeman merged commit 49c82ee into develop Mar 5, 2022
@olliefreeman olliefreeman deleted the feature/mc-9808 branch March 5, 2022 10:30
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

1 participant