Skip to content

4401 – Deprecate the Mafindo Parser#386

Merged
vasconsaurus merged 2 commits intodevelopfrom
4401-deprecate-mafindo
Apr 2, 2024
Merged

4401 – Deprecate the Mafindo Parser#386
vasconsaurus merged 2 commits intodevelopfrom
4401-deprecate-mafindo

Conversation

@vasconsaurus
Copy link
Contributor

Description

It's now using RSS/Check Zapier integration to import fact checks.

References: 4401

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can verify the changes. Please describe whether or not you implemented automated tests.

Things to pay attention to during code review

Please describe parts of the change that require extra attention during code review, for example:

  • File FFFF, line LL: This refactoring does this and this. Is it consistent with how it’s implemented elsewhere?
  • Etc.

Checklist

  • I have performed a self-review of my own code
  • I have added unit and feature tests, if the PR implements a new feature or otherwise would benefit from additional testing
  • I have added regression tests, if the PR fixes a bug
  • I have added logging, exception reporting, and custom tracing with any additional information required for debugging
  • I considered secure coding practices when writing this code. Any security concerns are noted above.
  • I have commented my code in hard-to-understand areas, if any
  • I have made needed changes to the README
  • My changes generate no new warnings

It's now using RSS/Check Zapier integration to import fact checks.
@vasconsaurus vasconsaurus requested review from DGaffney and caiosba April 2, 2024 12:05
@vasconsaurus vasconsaurus requested a review from caiosba April 2, 2024 12:33
@vasconsaurus vasconsaurus merged commit 728ee3b into develop Apr 2, 2024
@vasconsaurus vasconsaurus deleted the 4401-deprecate-mafindo branch April 2, 2024 19:03
@vasconsaurus
Copy link
Contributor Author

will deploy this once we have done Factly and Vishvas as well.

vasconsaurus added a commit that referenced this pull request Apr 18, 2024
It's now using RSS/Check Zapier integration to import fact checks.

Note: We also had to update a test that was using the mafindo parser.

Ticket: 4401
PR: #386
@vasconsaurus vasconsaurus mentioned this pull request Oct 24, 2025
8 tasks
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