open-reaction-database / ord-interface

Search/browse interface and APIs for the Open Reaction Database
https://open-reaction-database.org
Apache License 2.0
20 stars 11 forks source link

Fix reactions query in reaction viewer #132

Closed i404788 closed 3 months ago

i404788 commented 3 months ago

Should resolve #126

skearnes commented 3 months ago

I don't think this will work; the new API backend uses singular instead of plural names. I'll take a look at the issue today.

i404788 commented 3 months ago

@skearnes the code block I've edited calls https://github.com/open-reaction-database/ord-interface/blob/99bea17edd93832fec19ab1c901c94956c911a78/ord_interface/api/search.py#L182 which is the plural form; it's possible the single was intended, in that case the url needs to be changed, and list removed

skearnes commented 3 months ago

Oh sorry I was thinking of the query endpoint, not that one. Yes this fix is probably right then.

skearnes commented 3 months ago

Thanks @i404788; will deploy shortly.