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

Showdown - links without hash no longer work #58

Open
smalers opened this issue Aug 4, 2022 · 2 comments
Open

Showdown - links without hash no longer work #58

smalers opened this issue Aug 4, 2022 · 2 comments
Assignees
Labels
bug Something isn't working critical Priority: highest priority S Size: day or less

Comments

@smalers
Copy link
Contributor

smalers commented Aug 4, 2022

Now that the InfoMapper has been switched to path location strategy, I want to update markdown files that are used to not use has in the URL. However, this is not working, at least locally in the development environment. An old link looked like:

See also [***Release Notes***](#/content-page/release-notes).

A new link looks like the following (I tried with and without leading slash):

See also [***Release Notes***](/content-page/release-notes).
See also [***Release Notes***](content-page/release-notes).

The first link gives the following (note the duplicate content-page, but one is singular and one is plural):

image

The second link gives the following:

image

The good news is that the old hash location URLs do automatically convert to the new path location URLs. However, I need to know what the proper syntax is to include in Markdown content pages to work. This will also be used with all of the info popups.

It may also make sense in the application to handle converting hash to path and the other way around, depending on what the application uses. I'm not sure we'll ever go back to hash location strategy but don't want to lose track of how that might work. The default can be path location strategy.

@smalers smalers added bug Something isn't working critical Priority: highest priority S Size: day or less labels Aug 4, 2022
@Nightsphere
Copy link
Collaborator

Nightsphere commented Aug 5, 2022

There was a small bug in the code now that the hash is not being used, and that has been fixed. So all links will work at the moment, but because of the Showdown property openLinksInNewWindow being set, it considers internal links to other "pages" in the app to be external and will open it in a new tab as well.

I new issue has been created on the Showdown Github page and we are waiting to hear back from them.

@smalers
Copy link
Contributor Author

smalers commented Aug 5, 2022

Can you update this to link to the Showdown issue so I can monitor?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working critical Priority: highest priority S Size: day or less
Projects
None yet
Development

No branches or pull requests

2 participants