Details
-
Improvement
-
Resolution: Fixed
-
Neutral
-
1.2.0
-
None
Description
Please find some thoughts or improvement ideas below...
- Currently the target URL is rendered as external link. This is not really necessary as this URL shouldn't used as it is (see
VANITY-1). The page link can be rendered on time using the stored page ID. - External link rendering is only required for the QR code - if you want to save the QR code on the author instance.
- Currently the link building uses the default URL and removes the context path. This is quite hard coded. We have clients using different host names for author and public. They even have multiple public host names as working behind load balancer.
- It is fine if the current implementation works for you.
- But I would like to suggest making a URL prefix configurable on the Vaniry URL app.
Checklists
Acceptance criteria