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

Update Docker readme #4320

Merged
merged 5 commits into from
Apr 24, 2022
Merged

Update Docker readme #4320

merged 5 commits into from
Apr 24, 2022

Conversation

Alkarex
Copy link
Member

@Alkarex Alkarex commented Apr 18, 2022

#fix #3351
#fix #4308
#fix #2679

@gbakeman
Copy link

This is probably just a matter of personal choice, but I like to tie the container's timezone data with the host so everything is synced. I bind the following files:

  • /etc/localtime
  • /etc/timezone

Not sure if that's something you feel belongs in the example. Regardless, it looks great! Thank you.

@Alkarex
Copy link
Member Author

Alkarex commented Apr 24, 2022

Let's merge this one. Even it not perfect, it should already be much better than what we had before :-)

@Alkarex Alkarex merged commit 4e16dd1 into FreshRSS:edge Apr 24, 2022
@Alkarex Alkarex deleted the docker-doc branch April 24, 2022 16:35
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request May 30, 2022
Alkarex added a commit that referenced this pull request May 31, 2022
* Docker readme volume for extensions
Forgotten from #4320
Keeping #2837

* Minor whitespace

* No extension volume for development
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docker Everything related to Docker Documentation 📚
Projects
None yet
2 participants