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

[Datahub] Declutter the download section #1078

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ronitjadhav
Copy link
Collaborator

Description

This PR implements the filterByProtocol method to prioritize OGC API Features and avoid displaying redundant download links for the same format when available through multiple protocols. Only the OGC API Features version will be shown when both OGC API Features and another protocol provide the same format, ensuring a cleaner interface.

Additional Updates:

  • Added a new GeoNetwork record: "Zones de collecte de déchets en porte à porte - par flux de collecte, jour et horaire de tournée" from MEL.
  • Updated the database dump (docker-entrypoint-initdb.d/dump) to reflect the newly added record.

Screenshots

downloadDeclutter

Quality Assurance Checklist

  • Commit history is devoid of any merge commits and readable to facilitate reviews
  • If new logic ⚙️ is introduced: unit tests were added
  • If new user stories 🤏 are introduced: E2E tests were added
  • If new UI components 🕹️ are introduced: corresponding stories in Storybook were created
  • If breaking changes 🪚 are introduced: add the breaking change label
  • If bugs 🐞 are fixed: add the backport <release branch> label
  • The documentation website 📚 has received the love it deserves

Added a new record: 'Zones de collecte de déchets en porte à porte - par flux de collecte, jour et horaire de tournée' to the GeoNetwork database.
Copy link
Contributor

github-actions bot commented Jan 17, 2025

Affected libs: common-domain, api-metadata-converter, api-repository, feature-catalog, feature-record, feature-router, feature-search, feature-map, feature-editor, feature-auth, ui-search, feature-dataviz, common-fixtures, util-shared, ui-elements, feature-notifications, ui-catalog, ui-widgets, ui-inputs, ui-layout, ui-map, ui-dataviz, database-dump,
Affected apps: metadata-converter, metadata-editor, datahub, demo, webcomponents, map-viewer, search, datafeeder, data-platform,

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

Copy link
Contributor

github-actions bot commented Jan 17, 2025

📷 Screenshots are here!

@coveralls
Copy link

coveralls commented Jan 17, 2025

Coverage Status

coverage: 84.175% (-1.6%) from 85.755%
when pulling fbf7973 on declutter-downloads
into 411f5f1 on main.

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.

2 participants