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

Add Fulcro config #30

Merged
merged 3 commits into from
Sep 29, 2023
Merged

Add Fulcro config #30

merged 3 commits into from
Sep 29, 2023

Conversation

holyjak
Copy link
Contributor

@holyjak holyjak commented Sep 23, 2023

This includes two separate commits:

  1. b59650e Dev support - shadow-powered dev repl - see the commit for details. This is a proposal and may be dropped
  2. Initial support for Fulcro - see the commit for details

@holyjak holyjak marked this pull request as draft September 23, 2023 11:03
@holyjak holyjak force-pushed the add-fulcro branch 2 times, most recently from 9709e9e to 37282ad Compare September 24, 2023 10:48
@holyjak holyjak marked this pull request as ready for review September 24, 2023 10:53
@holyjak holyjak marked this pull request as draft September 27, 2023 22:21
@holyjak

This comment was marked as outdated.

@holyjak holyjak marked this pull request as ready for review September 28, 2023 07:18
Only few namespaces are exposed so far, but it is already
possible to render a trivial defsc component to a dom element.

FIXME: Tests currently don't run due to problems with accessing `js/ReactDOMServer`
Add :dev profile to deps, and shadow-cljs config to build the code and serve a page with it. The entrypoint is `dev/development.cljs`. See Readme.

This makes it easy to interactively develop and test SCI configs.
@holyjak holyjak marked this pull request as draft September 29, 2023 18:08
@holyjak
Copy link
Contributor Author

holyjak commented Sep 29, 2023

Made to draft, while troubleshooting state management in Fulcro.

@holyjak holyjak marked this pull request as ready for review September 29, 2023 18:52
@borkdude borkdude merged commit bf9769c into babashka:main Sep 29, 2023
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