Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RoutingHashReactHooks does not work in TryPureScript #273

Conversation

pete-murphy
Copy link
Collaborator

@pete-murphy pete-murphy commented Feb 8, 2021

This recipe causes odd behavior in TryPureScript (as first noted here: #271 (comment), and pictured below) where clicking on a link will re-direct such that the <iframe> renders a nested instance of TryPureScript. I've added a tryUnsupported.md and removed the Try link from the README table of contents. Interestingly this exact code will work if run as a gist in TryPureScript. This seems to be unexpected behavior; should I open an actual issue to track this?

2021-02-07 22 11 54

@JordanMartinez
Copy link
Owner

I want to say this is an issue with Try PureScript, but I'm not really sure what is causing it.

Either way, I think we should merge this to prevent this unexpected behavior.

@JordanMartinez JordanMartinez merged commit cb3aef7 into JordanMartinez:master Feb 9, 2021
@milesfrain
Copy link
Collaborator

try.ps.ai generates the iframe a bit differently than the original site, and this is definitely a regression. Thanks for reporting. I'll dig into this whenever I get around to making another pass at merging with upstream.

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.

None yet

3 participants