Skip to content

fix: specifying ic provider with a trailing slash is recognised correctly#2206

Merged
mergify[bot] merged 4 commits intomasterfrom
ic-provider-can-have-trailing-slash
May 16, 2022
Merged

fix: specifying ic provider with a trailing slash is recognised correctly#2206
mergify[bot] merged 4 commits intomasterfrom
ic-provider-can-have-trailing-slash

Conversation

@viviveevee
Copy link
Contributor

Description

User writes https://ic0.app/ instead of https://ic0.app. Dfx does not recognise the provider with a trailing slash as the real IC and tries to create a UI canister.

Fixes this problem a user encountered.

How Has This Been Tested?

Added a unit test.

Checklist:

  • The title of this PR complies with Conventional Commits.
  • I have edited the CHANGELOG accordingly.
  • I have made corresponding changes to the documentation.

@viviveevee viviveevee requested a review from a team as a code owner May 16, 2022 08:30
@mergify mergify bot merged commit 816a948 into master May 16, 2022
@mergify mergify bot deleted the ic-provider-can-have-trailing-slash branch May 16, 2022 17:32
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