@cope LMK you discord username and I can give you contributor badge in there if you dont already have one!

@cope LMK you discord username and I can give you contributor badge in there if you dont already have one!

😃

so, I made a mistake, here...

basically, I was at home over the weekend, on my personal comp, and not able to VPN (don't ask) so I was unable to verify that my company custom URL is also using {subdomain}.{domain}.{tld}/wiki/spaces/{spaceKey}/*... I stupidely assumed they are. (how often do you examine long URLs, frankly)

which, it turns out, we are not 😭

instead, for some reason, our IT clearly had ideas, so we are using {subdomain}.{domain}.{tld}/display/{spaceKey}/*, instead...
kill me...

@timothycarambat do you think we can solve this, somehow?

I do not think that adding the explicit handling for /display/ makes sense, tbh. It is way to specific...

But, maybe if the match fails, we try to use the passed URL as is, assuming it is a proper SPACE URL and not just any PAGE url? The ConfluencePagesLoader will throw an error for a non-space URL, this I have confirmed already.

Alternatively, I would have to scrape all the pages myself and push them through the /document/upload API...

It looks like the display is not specific to your IT team, I think it is actually how they do human-readable names.

If it makes no sense, is clunky, and leads to a bad DX you can guarantee it's an Atlassian product:
https://confluence.atlassian.com/doc/space-keys-829076188.html

So adding an edge case for that display URL is not crazy it appears. We can either modify the pattern or make it a function that basically runs a list of patterns and returns on first match

OH, awesome! PR comming up 😄

Created 1423

@timothycarambat we forgot to adjust line 64: baseUrl = 'https://${customDomain}/wiki';

In a human readable form, the baseUrl needs to be only https://${customDomain}, so I had to distinguish between the two...

Created 1434