Skip to content

(fix) Add gateway URI to deploy.yaml template on interface#1381

Merged
dOrgJelli merged 1 commit into
polywrap:origin-0.9-devfrom
sraver:fix-interface-build-template
Nov 2, 2022
Merged

(fix) Add gateway URI to deploy.yaml template on interface#1381
dOrgJelli merged 1 commit into
polywrap:origin-0.9-devfrom
sraver:fix-interface-build-template

Conversation

@sraver
Copy link
Copy Markdown
Contributor

@sraver sraver commented Nov 2, 2022

I've found that when creating a new interface from the template there was a missing config that didn't allow me to deploy.

I've added the new config parameter and the required step on the README.

Copy link
Copy Markdown
Contributor

@Niraj-Kamdar Niraj-Kamdar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks @sraver for catching this and contributing!

Copy link
Copy Markdown
Contributor

@pileks pileks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Nice catch!

Are we pushing this into 0.10-dev too? 😄

@sraver
Copy link
Copy Markdown
Contributor Author

sraver commented Nov 2, 2022

LGTM! Nice catch!

Are we pushing this into 0.10-dev too? smile

Sure! I can create another PR for the 0.10, but i guess it depends on the gitflow you using. Normally once the 0.9-dev gets deployed, the 0.9 could be merged into the 0.10-dev to bring all the fixes that have been deployed there.

@dOrgJelli dOrgJelli merged commit 03906ef into polywrap:origin-0.9-dev Nov 2, 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.

4 participants