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

Packaging, new filter options and syntax check #10

Open
wants to merge 54 commits into
base: master
Choose a base branch
from
Open

Conversation

kkaris
Copy link
Contributor

@kkaris kkaris commented Feb 6, 2025

This PR does four things:

  • Checking the syntax of the curation comment depending on source. If activated, the syntax check is done at submission time and returns a 422 with error text describing the issue if the text is invalid.
  • Filter out already curated statements. This resolves Flag for filtering statements that already have curations #8.
  • Search box that filters based in the english assembled text of each statement.
  • Restructure repository for packaging

This PR also includes minor updates to Vue/JS code to remove warnings and errors as well as some smaller style fixes.

@kkaris kkaris requested a review from bgyori February 6, 2025 00:47
@kkaris kkaris self-assigned this Feb 6, 2025
@kkaris kkaris changed the title Signor comment text More options Feb 6, 2025
@kkaris kkaris linked an issue Feb 6, 2025 that may be closed by this pull request
@kkaris kkaris changed the title More options Packaging, new filter options and syntax check Feb 11, 2025
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.

Flag for filtering statements that already have curations
3 participants