Skip to content

Look explicitly into JSON payload vs .params#258

Merged
snacktavish merged 1 commit intomasterfrom
fix_include_exclude_trees
Oct 24, 2023
Merged

Look explicitly into JSON payload vs .params#258
snacktavish merged 1 commit intomasterfrom
fix_include_exclude_trees

Conversation

@jimallman
Copy link
Copy Markdown
Member

@jimallman jimallman commented Oct 24, 2023

This should fix the include_tree_in_synth and exclude_tree_from_synth methods on devtree.

@jimallman
Copy link
Copy Markdown
Member Author

jimallman commented Oct 24, 2023

Confirmed, this is working now on devtree, see for example https://devtree.opentreeoflife.org/curator/study/edit/pg_2823?tab=home
It's a tiny pull request and should be easy to merge+deploy on master.

@snacktavish snacktavish merged commit a62494d into master Oct 24, 2023
@snacktavish
Copy link
Copy Markdown
Member

@jimallman Seems to be working on dev.... but despite update to production is not working there.
e.g. https://tree.opentreeoflife.org/curator/study/edit/ot_2262/?tab=home

@jimallman
Copy link
Copy Markdown
Member Author

Interesting! I get a mysterious error on API response, but if I refresh the page, the page has apparently been included/excluded as it should. Will investigate further...

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