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

fix(ME): do not return null on text fields for new records #965

Merged
merged 5 commits into from
Aug 22, 2024

Conversation

jahow
Copy link
Collaborator

@jahow jahow commented Aug 22, 2024

This was causing some mandatory text fields (abstract and lineage) to become null after a draft was saved for the first time.

Copy link
Contributor

github-actions bot commented Aug 22, 2024

Affected libs: api-metadata-converter, feature-search, feature-router, feature-map, feature-dataviz, feature-record, api-repository, feature-catalog, feature-editor, feature-auth, ui-search,
Affected apps: metadata-editor, metadata-converter, datahub, demo, webcomponents, map-viewer, search, datafeeder,

  • 🚀 Build and deploy storybook and demo on GitHub Pages
  • 📦 Build and push affected docker images

@coveralls
Copy link

coveralls commented Aug 22, 2024

Coverage Status

coverage: 83.588% (+1.9%) from 81.672%
when pulling 621b04f on ME-fix-new-record-crash
into 38f835a on main.

Copy link
Contributor

github-actions bot commented Aug 22, 2024

📷 Screenshots are here!

@jahow jahow merged commit 825a487 into main Aug 22, 2024
11 checks passed
@jahow jahow deleted the ME-fix-new-record-crash branch August 22, 2024 10:10
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.

3 participants