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

reflex-dom 0.6.3.1 uses unreleasead reflex-dom-core function #462

Closed
maralorn opened this issue Sep 17, 2023 · 1 comment
Closed

reflex-dom 0.6.3.1 uses unreleasead reflex-dom-core function #462

maralorn opened this issue Sep 17, 2023 · 1 comment

Comments

@maralorn
Copy link
Contributor

While building reflex-dom 0.6.3.1 in nixpkgs I get:

reflex-dom> [3 of 3] Compiling Reflex.Dom.Location.Platform ( src/Reflex/Dom/Location/Platform.hs, dist/build/Reflex/Dom/Location/Platform.o, dist/build/Reflex/Dom/Location/Platform.dyn_o )
reflex-dom> 
reflex-dom> src/Reflex/Dom/Location/Platform.hs:16:38-72: error:
reflex-dom>     Not in scope: ‘Reflex.Dom.Location.popHistoryState’
reflex-dom>     NB: the module ‘Reflex.Dom.Location’ does not export ‘popHistoryState’.
reflex-dom>     Suggested fix:
reflex-dom>       Perhaps use one of these:
reflex-dom>         data constructor ‘Reflex.Dom.Location.HistoryItem’ (imported from Reflex.Dom.Location),
reflex-dom>         data constructor ‘Reflex.Dom.Location.HistoryStateUpdate’ (imported from Reflex.Dom.Location)
reflex-dom>    |
reflex-dom> 16 | module Reflex.Dom.Location.Platform (Reflex.Dom.Location.popHistoryState) where
reflex-dom>    |   

Reflex.Dom.Location.popHistoryState should be exported from reflex-dom-core. But the newest release of reflex-dom-core 0.8.0.0 does not export it. The change that moved the function from reflex-dom to reflex-dom-core happened in #453 after the reflex-dom-core release 0.8.0.0 but before the last reflex-dom release.

@ali-abrar
Copy link
Member

Thanks for the report. reflex-dom-core 0.8.1.0 has been released with those functions, and reflex-dom 0.6.3.1 has been revised on hackage to reflect this version dependency.

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

No branches or pull requests

2 participants