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

Remove Docker references from documentation and code #7648

Closed
2 tasks
BOHEUS opened this issue Oct 13, 2024 · 5 comments
Closed
2 tasks

Remove Docker references from documentation and code #7648

BOHEUS opened this issue Oct 13, 2024 · 5 comments

Comments

@BOHEUS
Copy link
Contributor

BOHEUS commented Oct 13, 2024

As core team decided not to support Docker development containers here thanks to many tickets in #help channel on Discord server and general problems in maintaining, we should clean this before next release. Only Docker dev containers should be affected, Postgres and Redis container are still supported.

Tasks

Preview Give feedback
@BOHEUS
Copy link
Contributor Author

BOHEUS commented Oct 13, 2024

@charlesBochet did I miss something? If you want, I can handle this

@charlesBochet
Copy link
Member

@BOHEUS I think we should keep them but make clear they are community maintained.
I think we can get rid of:

  • twenty-docker/twenty-website
  • Makefile

but I would keep the other ones.

Regarding the docs, we should remove the references to docker except for self-hosting + postgres local container. I'm not sure there are actually

@charlesBochet
Copy link
Member

I think the mindset of these dockers config is that they can be used for production hosting (production built container). We don't want to support docker for local setup (to contribute).

So the Makefile does not make sense + twenty-website does not makes sense

  • twenty is still useful for self-host
  • postgres + postgres-spilo are useful to get postgres for self-host (and local contribution for this one as we don't need a dev specific container for postgres)

@BOHEUS
Copy link
Contributor Author

BOHEUS commented Oct 13, 2024

Okay, I'll handle this and create a PR soon

@charlesBochet
Copy link
Member

TY!

@BOHEUS BOHEUS mentioned this issue Oct 13, 2024
charlesBochet pushed a commit that referenced this issue Oct 13, 2024
Related to #7648 

@charlesBochet I've deleted only dev related commands in Makefile, if
it's wrong, I'll change it ASAP
@BOHEUS BOHEUS closed this as completed Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants