BeaconCMS / beacon

Open-source content management system (CMS) built with Phoenix LiveView. Faster render times to boost SEO performance, even for the most content-heavy pages.
https://beaconcms.org
MIT License
1.04k stars 101 forks source link

Use a `DynamicSupervisor` for `Beacon` #651

Closed leandrocp closed 6 days ago

leandrocp commented 1 week ago

Migrate from a traditional Supervisor to DynamicSupervisor so we have more control to manage sites (children).

APB9785 commented 6 days ago

Regular Supervisor ends up fitting our use-case better here, as the site processes aren't temporary (using DynamicSupervisor implies that its children are temporary processes), and we still do have the possibility of Supervisor.terminate_child/2 and Supervisor.restart_child/2 if needed