coollabsio
feat(service): add Opnform template
Changes
- Add new template to deploy opnform app with coolify (see https://opnform.com/)
Hey @ShadowArcanist @peaklabs-dev anything else for us to do here are we good? thanks!
Hey @JhumanJ I don’t have control over merging PRs in this repository, so I’m unable to provide a definitive update on the status of this one.
The decision ultimately rests with the Core Maintainers. I did notice that Peak, one of the maintainers, added a "Hold" label to the PR, which might suggest they’re currently prioritizing other tasks — but I can’t say for certain.
Hi @alexbaron-dev! 👋
It appears to us that you are either adding a new service or making changes to an existing one. We kindly ask you to also review and update the Coolify Documentation to include this new service or it's new configuration needs. This will help ensure that our documentation remains accurate and up-to-date for all users.
Coolify Docs Repository: https://github.com/coollabsio/coolify-docs How to Contribute a new Service to the Docs: https://coolify.io/docs/get-started/contribute/service#adding-a-new-service-template-to-the-coolify-documentation
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Hello, I have updated files as requested and add documentation (see https://github.com/coollabsio/coolify-docs/pull/401). I hope everything fine now :-)
It works for me even tho the generated url start by https://nginx-gggssc84s0515e4cc8o4c0.example.com)
You could maybe rename the SERVICE_URL_NGINX to SERVICE_URL_OPNFORM even if it is not in the envs of opnform container, to avoid confusion...
Hey @yipfram thanks for approving. Issue still have waiting for changes label, can we please get this changed?
When we can expect this to be merged?
Thanks
Hey @yipfram thanks for approving. Issue still have waiting for changes label, can we please get this changed? When we can expect this to be merged? Thanks
Thank you for your submission. I wasn't able to approve the changes yet because the suggestion mentioned here has not been implemented. This suggestion ensures consistency in formatting the environment variables, in line with other templates in the repository. Once that change has been made, I will be happy to test the template and, assuming everything works as expected, mark the PR as ready to merge.
Thanks for getting back to us @ShadowArcanist, I have just updated the template again
Hate to be chasing @ShadowArcanist, but we'd really like to get this merged after all these months 🙏 I'm available if anything else needs to be done here - thanks!
Hate to be chasing @ShadowArcanist, but we'd really like to get this merged after all these months 🙏 I'm available if anything else needs to be done here - thanks!
Hey, I will get back to this in next 48hrs, currently I have lot going on. The template looks good to me but I just have to test it on my dev coolify instance.
Thank you so much @ShadowArcanist for your time!
I just updated the templates, replacing SERVICE_FQDN_NGINX with SERVICE_URL_NGINX which fixed the broken links when opening the form from the app. Also removed a variable that's not needed anymore on nginx.
I ran some more tests changing the domain and it's working just fine.
Thanks again!
Thank you so much @ShadowArcanist for your time! I just updated the templates, replacing
SERVICE_FQDN_NGINXwithSERVICE_URL_NGINXwhich fixed the broken links when opening the form from the app. Also removed a variable that's not needed anymore on nginx. I ran some more tests changing the domain and it's working just fine. Thanks again!
Keep the SERVICE_URL_NGINX under the nginx service, right now Coolify doesn't autognerate domain so users might think the template is broken:
It will show the urls if you are just editing the template via the edit compose feature but new deployments won't see the urls
Hey @ShadowArcanist I just bumped image version if you can please take a look 🙏
Also, @peaklabs-dev when can we hope to get this merged? Thanks!