Skip to content

Conversation

@rlaveycal
Copy link
Contributor

There are absolute path references in links that stop the UI from working if hosted in IIS at a non-root location.

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

CLAassistant commented Nov 17, 2020

CLA assistant check
All committers have signed the CLA.

@WojcikMike
Copy link
Contributor

Thanks @rlaveycal for fixing this for us. I will take a look at some automatic way for checking it. As this bug sneaks in more often than it should.

@WojcikMike WojcikMike self-assigned this Nov 18, 2020
@WojcikMike WojcikMike added this to the 1.28.0 milestone Nov 18, 2020
@WojcikMike WojcikMike changed the base branch from master to external-contributions November 18, 2020 12:57
@WojcikMike WojcikMike merged commit c353004 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>
@WojcikMike WojcikMike added the Type: Bug Type: Bug label Nov 19, 2020
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: Bug Type: Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants