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

Fix a typo in DevReadMe.md #1844

Merged
merged 3 commits into from
Jun 30, 2022
Merged

Conversation

feknall
Copy link
Contributor

@feknall feknall commented Jun 25, 2022

There is a typo in the example command to run the project using the docker container.

PORTS="5000:5000 8000:8000 1000:1000" ./scripts/run_docker start --inbound-transport http 0.0.0.0 10000 --outbound-transport http --debug --log-level DEBUG

It opens the port 1,000 for access instead of 10,000 which is specified as "inbound-transport". As a result, the agent wouldn't be accessible.

@swcurran
Copy link
Contributor

Thanks for this -- much appreciated!! We need all commits signed using DCO in order to merge. Could you update your commit to sign it: DCO - Developer Certificate of Origin - https://github.com/apps/dco -- see the instructions on the "Details" link on the DCO line above ^^^^^, Thanks!

@feknall
Copy link
Contributor Author

feknall commented Jun 28, 2022

You're welcome. It's done. Please check it again.

@swcurran swcurran merged commit 2a33785 into openwallet-foundation:main Jun 30, 2022
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.

2 participants