Skip to content

Issue #810 Remove spaces in the MDR entities' and attributes' fields of name and uniqueName#868

Open
thelmick-unicon wants to merge 1 commit intoLIF-Initiative:mainfrom
thelmick-unicon:main
Open

Issue #810 Remove spaces in the MDR entities' and attributes' fields of name and uniqueName#868
thelmick-unicon wants to merge 1 commit intoLIF-Initiative:mainfrom
thelmick-unicon:main

Conversation

@thelmick-unicon
Copy link
Contributor

Fixes

Description of Change

Include:

  • What problem does this solve?
    Spaces, and other characters, in the MDR entities' and attributes' fields of name and uniqueName break the transforms.

  • What is the solution?
    Removing the offending characters from the MDR data.

  • How should reviewers test this?
    Review the data to see if any spaces, commas, colons, or tilde are in the dataset.

Related Issues

#801

Closes # [[add Github issue number]]
#810

Type of Change
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality
    to not work as expected)
  • Documentation update
  • Infrastructure/deployment change
  • Performance improvement
  • Code refactoring
Project Area(s) Affected
  • bases/
  • components/
  • orchestrators/
  • frontends/
  • deployments/
  • CloudFormation/SAM templates
  • Database schema
  • API endpoints
  • Documentation
  • Testing

…butes' fields of name and uniqueName

Fixes
Copy link
Contributor

@cbeach47 cbeach47 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The openapi diff is pretty hard to see the differences - are the only changes in the openapi json file the adjusted Name / UniqueName fields after the database changes?

For the Flyway migration files - V1.2_... should be the just UPDATEs to create the deltas. The migration tool should recognize that V1.2_... hasn't been applied yet, and run the UPDATEs. I would rename V1.1_to_V1.2_delta.sql to V1.2__remove_reserved_characters.sql, and then remove the V1.2__metadata_repository_init.sql file.

Are you confident these entity / attribute IDs will be the same in Dev and Demo as they are in Local?

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

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants