1715labs / 1715labs-pfe

Modified version of PFE for use with paid crowds
1715labs-pfe.vercel.app
Apache License 2.0
0 stars 0 forks source link

change default to have workflow querying enabled #18

Open colin1715labs opened 3 years ago

colin1715labs commented 3 years ago

Currently workflow querying has to be enabled for each project - this has caused a number of hiccups when it has not been activated. Going forwards nearly all projects will need workflow querying enabled, to allow for AAAPI workflow redirection.

Can we updated our version of PFE to have workflow querying enabled by default?

tingard commented 3 years ago

I think we shouldn't even have the check here - if a workflow is in the url then always go to it

tingard commented 2 years ago

This doesn't seem to have worked - for instance keep refreshing this page and you will be bumped around between workflows A, B and C, despite the workflow ID being in the URL https://pfe-staging.1715labs.com/projects/1715labs/workflow-query-example-project/classify?workflow=19761

tingard commented 2 years ago

~Apologies - I'm still using the old PFE URL not vercel!!~ didn't realise the vercel app points to pfe-staging.1715labs.com

tingard commented 2 years ago

Not sure why I was seeing the jumping-around bug though, it seems to work now... Could the changes have been deployed to prod but not staging?