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

Database / Default content / Remove categories. #8063

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fxprunayre
Copy link
Member

@fxprunayre fxprunayre commented May 17, 2024

The default categories provided in GeoNetwork since the early beginning are quite overlapping hierarchy level in ISO (eg. dataset, application, map) and in practice they are never used with the default values. It was making sense at the start to have a common grouping systems for the initial standards available in GeoNetwork like dublin core, FGDC, iso19115. Nowadays it is not recommended to use this default category list.

Here we propose to not populate the default database with any category (and we do not alter databases which may be using them).

  • Before

image

  • After

image

This will also keep the UI simple. eg. If there is no category, no menu available in the editor (so users are not tempted to use the non recommended default values provided).

image

We should highlight that the main (only) benefit of using category mechanism is to group records without adding the information in the metadata document. The documentation was updated.

Checklist

  • I have read the contribution guidelines
  • Pull request provided for main branch, backports managed with label
  • Good housekeeping of code, cleaning up comments, tests, and documentation
  • Clean commit history broken into understandable chucks, avoiding big commits with hundreds of files, cautious of reformatting and whitespace changes
  • Clean commit messages, longer verbose messages are encouraged
  • API Changes are identified in commit messages
  • Testing provided for features or enhancements using automatic tests
  • User documentation provided for new features or enhancements in manual
  • Build documentation provided for development instructions in README.md files
  • Library management using pom.xml dependency management. Update build documentation with intended library use and library tutorials or documentation

@fxprunayre fxprunayre added db change Indicate that this work introduces a db structure change. changelog labels May 17, 2024
@fxprunayre fxprunayre added this to the 4.4.5 milestone May 17, 2024
@fxprunayre fxprunayre force-pushed the 44-categoriescleanup branch 2 times, most recently from 0981f5c to e1b2f72 Compare May 17, 2024 09:30
The default categories provided in GeoNetwork since the early beginning are quite overlapping hierarchy level in ISO (eg. dataset, application, map) and in practice they are never used with the default values. It was making sense at the start to have a common grouping systems for the initial standards available in GeoNetwork like dublin core, FGDC, iso19115. Nowadays it is not recommended to use this default category list.

Here we propose to not populate the default database with any category (and we do not alter databases which may be using them).
This will also keep the UI simple. eg. If there is no category, no menu available in the editor (so users are not tempted to use the non recommended default values provided).

We should highlight that the main (only) benefit of using category mechanism is to group records without adding the information in the metadata document.
Copy link

sonarcloud bot commented May 17, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@fxprunayre fxprunayre modified the milestones: 4.4.5, 4.4.6 Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog db change Indicate that this work introduces a db structure change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant