Skip to content

Add permits for index entries#454

Merged
MarekSuchanek merged 1 commit intodevelopfrom
feature/index-filtering
Oct 31, 2023
Merged

Add permits for index entries#454
MarekSuchanek merged 1 commit intodevelopfrom
feature/index-filtering

Conversation

@MarekSuchanek
Copy link
Copy Markdown
Contributor

@MarekSuchanek MarekSuchanek commented Sep 5, 2023

This pull request introduces a moderation mechanism for index entries by adding support for permit states. Index entries, which are automatically created when any service pings the FDP index, can now be reviewed and assigned a state: PENDING, ACCEPTED, or REJECTED. This allows more control over what gets publicly listed and prevents unwanted or invalid records from appearing in the index without review.

  • Enables moderation: Entries can now be manually reviewed before becoming publicly visible.
  • Improves trust and quality: Ensures only approved entries appear publicly in the index.
  • Prevents spam or invalid data: Automatically created entries are no longer blindly accepted (unless configured so).
  • Clear state management: States (PENDING, ACCEPTED, REJECTED) make it easy to track and act on entries.

@MarekSuchanek MarekSuchanek added the feature Request for new functionality label Sep 5, 2023
@MarekSuchanek MarekSuchanek self-assigned this Sep 5, 2023
@MarekSuchanek MarekSuchanek force-pushed the feature/index-filtering branch from d8a86fe to 79892e1 Compare September 5, 2023 13:34
@MarekSuchanek MarekSuchanek force-pushed the feature/index-filtering branch from 79892e1 to 5cbcc61 Compare September 6, 2023 07:00
Copy link
Copy Markdown
Contributor

@kburger kburger left a comment

Choose a reason for hiding this comment

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

Minor question inline

@MarekSuchanek MarekSuchanek force-pushed the feature/index-filtering branch 11 times, most recently from 63b43da to e740014 Compare September 15, 2023 11:51
@MarekSuchanek MarekSuchanek force-pushed the feature/index-filtering branch 2 times, most recently from ec09d57 to 69d44dd Compare September 15, 2023 13:40
@MarekSuchanek MarekSuchanek force-pushed the feature/index-filtering branch from 69d44dd to 97ff0cf Compare September 15, 2023 14:18
@MarekSuchanek MarekSuchanek force-pushed the feature/index-filtering branch from 97ff0cf to f169fff Compare October 3, 2023 06:00
@MarekSuchanek MarekSuchanek force-pushed the feature/index-filtering branch from f169fff to a8c86ea Compare October 3, 2023 06:11
@MarekSuchanek MarekSuchanek requested a review from kburger October 3, 2023 06:45
@MarekSuchanek
Copy link
Copy Markdown
Contributor Author

@kburger I added the configuration default value for auto-permit and also actions on permit change (deleting harvested data and triggering retrieval if accepted). Could you please check one more time so we can merge this?

@MarekSuchanek MarekSuchanek marked this pull request as ready for review October 3, 2023 06:46
@MarekSuchanek MarekSuchanek merged commit 172c2c4 into develop Oct 31, 2023
@MarekSuchanek MarekSuchanek deleted the feature/index-filtering branch October 31, 2023 12:46
@dennisvang
Copy link
Copy Markdown
Contributor

Is this related to FAIRDataTeam/jira-backlog-migration#32?

@dennisvang dennisvang linked an issue May 19, 2025 that may be closed by this pull request
8 tasks
@dennisvang
Copy link
Copy Markdown
Contributor

dennisvang commented May 19, 2025

Note the present PR (#454) predates the move to postgres (#519), so it still uses mongodb.
This implies it should be relatively easy to backport to v1.17.x

The following related issues were created after the move to postgres:

In case of a backport to v1.17.x, we should check if any of these issues also apply to the present PR.

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

Labels

feature Request for new functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FDP Index gatekeeper

4 participants