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

auto reload sql queries in dev mode #128

Closed
yogthos opened this issue Feb 2, 2024 · 7 comments
Closed

auto reload sql queries in dev mode #128

yogthos opened this issue Feb 2, 2024 · 7 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@yogthos
Copy link
Collaborator

yogthos commented Feb 2, 2024

Currently, whenever hugsql queries are changed, the user has to run (reset) to load new queries. Proposed change would be to automatically reload queries when the app is running in dev mode so that (reset) isn't needed.

This issue relates to discussion regarding automatically reloading routes in dev.

@yogthos yogthos added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Feb 2, 2024
@gerdint
Copy link
Contributor

gerdint commented Feb 20, 2024

I will have a go at this!

@gerdint
Copy link
Contributor

gerdint commented Feb 26, 2024

I would still want to have a go at this but it is subject to my available time which appears scarce for the time being, so I wouldn't want to block anyone else who is so inclined!

@yogthos
Copy link
Collaborator Author

yogthos commented Feb 26, 2024

Oh yeah no worries, and absolutely no rush. I haven't really had much time myself either lately. :)

@vollcheck
Copy link
Contributor

vollcheck commented Sep 2, 2024

Hey, I've implemented mentioned functionality - at least for the kit-sql-conman library.
@yogthos
#145

@gerdint
Copy link
Contributor

gerdint commented Sep 3, 2024

@vollcheck Thanks for working on this!

@vollcheck
Copy link
Contributor

vollcheck commented Sep 3, 2024

@gerdint no problem ;)

@yogthos I believe that issue can be closed now as #145 is merged already.

@yogthos
Copy link
Collaborator Author

yogthos commented Sep 3, 2024

and I just pushed out the new version to clojars, thanks again for spearheading this :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants