search

openwisp / docker-openwisp

OpenWISP in docker. For production usage we recommend using the ansible-openwisp2 role.
https://openwisp.io/docs/dev/docker/
BSD 3-Clause "New" or "Revised" License
149 stars 75 forks source link

Some updates #283

Closed katomaso closed 3 months ago

katomaso commented 1 year ago
nemesifier commented 3 months ago

Thank you for your proposal, @katomaso. I appreciate your effort, and I was able to incorporate some aspects of it when I recently updated the image. However, I regret to inform you that we cannot merge the rest of the patch as it currently stands.

The patch attempts to address multiple potential issues that haven't been clearly defined. Additionally, it modifies numerous files, which makes it difficult for us to confidently merge the changes without the risk of introducing new problems.

If you are still using OpenWISP and are interested in getting your changes merged, I suggest breaking your contributions into smaller, focused patches. Each patch should address a specific issue or a related group of issues and include a clear explanation of the problem being solved. This approach will simplify our review process and increase our confidence in merging your changes.

Thank you again for your contributions, and I look forward to seeing more from you in the future.