Skip to content

Add Wikidata route to create iNat taxon on Wikidata - #31

Draft
medied wants to merge 1 commit into
lubianat:masterfrom
medied:wikidata-route
Draft

Add Wikidata route to create iNat taxon on Wikidata#31
medied wants to merge 1 commit into
lubianat:masterfrom
medied:wikidata-route

Conversation

@medied

@medied medied commented Apr 2, 2025

Copy link
Copy Markdown

🚧 Work in progress

This PR adds a new route to handle the case of some taxa being available on iNaturalist but not registered on Wikidata with its corresponding property. The general user flow begins by providing the scientific name and then the app checks whether it is already present on Wikidata; if not, an option is provided to add taxon ID to Wikidata.

Addresses #29

@lubianat

lubianat commented Apr 3, 2025

Copy link
Copy Markdown
Owner

@medied thank you for the PR!

As you may have seen, the code base here is a bit messy (a large scale refactor/rewrite is overdue). That said, I'll be happy to review this PR, probably on a couple of weeks, though.

Worst case scenario, I am reviewing it at the Wikimedia Hackathon (https://www.mediawiki.org/wiki/Wikimedia_Hackathon_2025/Participants) in about 1 month! Sorry about the long time, though!

@medied

medied commented Apr 8, 2025

Copy link
Copy Markdown
Author

Sounds good, no rush at all! It's a shame I can't make it to the hackathon this year, but either way we can connect then to go over questions if needed.

@lubianat

lubianat commented May 2, 2025

Copy link
Copy Markdown
Owner

Hi, @medied, I am rewriting this codebase. I have extracted the package/module to https://github.com/lubianat/inat2wiki-module, where I'll maintain it independently, add some tests/CI, and make it a bit more enjoyable to change.

There will be a decent amount of changes, and I hope to feed your PR into it ASAP, but ASAP may take a while! Sorry and thanks again!

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