Skip to content

fix docker run command in README.md - #2

Open
tkothe wants to merge 1 commit into
xme:masterfrom
tkothe:patch-1
Open

fix docker run command in README.md#2
tkothe wants to merge 1 commit into
xme:masterfrom
tkothe:patch-1

Conversation

@tkothe

@tkothe tkothe commented Apr 15, 2019

Copy link
Copy Markdown

docker run was missing rootshell prefix when referencing the image to run.

docker run was missing `rootshell` prefix when referencing the image to run.
@tkothe

tkothe commented Apr 15, 2019

Copy link
Copy Markdown
Author

we should probably update this even more to expose the 443 port. Without that, it won't be usable from the host machine. What do you think @xme ?

@tkothe

tkothe commented Apr 15, 2019

Copy link
Copy Markdown
Author

and on a completely unrelated note: unfortunately for me this solution is not working. I was trying to convert the emails that Uber currently sends as their invoice emails to PNG. They don't convert well with this wkhtmltoimage based solution. It basically only shows the map of the Uber ride
Downloads

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant