Skip to content

Conversation

@RC-787
Copy link
Contributor

@RC-787 RC-787 commented Nov 16, 2020

Currently, the string "put here the url we tried to connect to" is displayed as part of the error message that is shown when unable to connect to ServiceControl. This change shows the actual ServiceControlUrl

@CLAassistant
Copy link

CLAassistant commented Nov 16, 2020

CLA assistant check
All committers have signed the CLA.

@WilliamBZA WilliamBZA added this to the 1.28.0 milestone Nov 18, 2020
@WilliamBZA WilliamBZA added the Type: Improvement Type: Improvement label Nov 18, 2020
@WilliamBZA
Copy link
Member

Thanks @RC-787!

@WojcikMike WojcikMike changed the base branch from master to external-contributions November 18, 2020 12:57
Co-authored-by: Sergio <sergioc@users.noreply.github.com>
@WojcikMike WojcikMike merged commit 4497c60 into Particular:external-contributions Nov 18, 2020
WojcikMike pushed a commit that referenced this pull request Nov 18, 2020
* replace absolute paths with relative (#985)

Signed-off-by: Richard Lavey <richard.lavey@calastone.com>

* Show ServiceControlUrl in error message when unable to connect (#983)

* Show ServiceControlUrl in error message when unable to connect

* Update src/ServicePulse.Host/app/js/views/sc_not_available.html

Co-authored-by: Sergio <sergioc@users.noreply.github.com>

Co-authored-by: Michał Wójcik <michal.wojcik@particular.net>
Co-authored-by: Sergio <sergioc@users.noreply.github.com>

Co-authored-by: Richard Lavey <richard.lavey@calastone.com>
Co-authored-by: RC-787 <58704461+RC-787@users.noreply.github.com>
Co-authored-by: Sergio <sergioc@users.noreply.github.com>
WilliamBZA pushed a commit that referenced this pull request Feb 17, 2023
* replace absolute paths with relative (#985)

Signed-off-by: Richard Lavey <richard.lavey@calastone.com>

* Show ServiceControlUrl in error message when unable to connect (#983)

* Show ServiceControlUrl in error message when unable to connect

* Update src/ServicePulse.Host/app/js/views/sc_not_available.html

Co-authored-by: Sergio <sergioc@users.noreply.github.com>

Co-authored-by: Michał Wójcik <michal.wojcik@particular.net>
Co-authored-by: Sergio <sergioc@users.noreply.github.com>

Co-authored-by: Richard Lavey <richard.lavey@calastone.com>
Co-authored-by: RC-787 <58704461+RC-787@users.noreply.github.com>
Co-authored-by: Sergio <sergioc@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Improvement Type: Improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants