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

Fix for relative links #181

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jack-fireworkhq
Copy link

@jack-fireworkhq jack-fireworkhq commented Apr 29, 2022

Hi Kovetskiy, First of all, thank you for creating this great repo.

I have done some effort to resolve #55, I think the main cause for this is here which makes Mark failed to find relative links.
I fixed it with code here.

Besides that, I add a "-L" option so that Mark can take many files at the same time as input.
I did that to let users be able to synchronize files references with relative links to each other.
I add a loop here to retry resolve relative links after everything was synchronized to Confluence once.

I tested this with the Github action I forked and modified it, and Mark works as my expectation.
Leave some comments if you feel some code needs to be changed.

PS: This JackTheMico is my personal account, I guess there is a misconfiguration of my git 😄 .

@kertechs
Copy link

I had the problem with relative links
I made a quick test (confluence cloud 1000.0.0-a50a53c8c11e) and it solved the proble for me (the links were not recognized/displayed by confluence)

Thanks to all of you for your work and help

@adrianrussell
Copy link

@jack-fireworkhq , @kertechs is this PR good to be merged ?

@jack-fireworkhq
Copy link
Author

@jack-fireworkhq , @kertechs is this PR good to be merged ?

It works for me and my team. I'd recommend you try it first before merging it

@nyarly
Copy link
Contributor

nyarly commented Jan 9, 2023

I'd be eager to have this feature merged

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.

Missing documentation regarding git based links and how they are treated when converted to Confluence page
5 participants