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

Select primary site alias by default when adding a new page url #3750

Closed
schotman opened this issue May 8, 2020 · 1 comment
Closed

Select primary site alias by default when adding a new page url #3750

schotman opened this issue May 8, 2020 · 1 comment

Comments

@schotman
Copy link
Contributor

schotman commented May 8, 2020

Description of problem

This isssue and (related pull request) makes life easier for a content manager and prevents problems.
When a content manager wants to ad a custom url for a page, he must select the site alias and enter the new custom url. Selecting a site alias, is usually not something he/she is accustomed to (they may even be unaware of the concept).
When there are multiple site aliasses and the first site alias is not (one of) the active/live alias(ses), then adding a new 200 page url can lead to a situation that the page can no longer be visisted (when the content manager forgets to select the proper site alias) because the custom 200 url links to the wrong site alias and results in a 404.

Description of solution

Currently the UI defaults to the first alias from the portal alias table (if the alias with id=1 is no longer there, the box is empty). Better would be if the primary alias would be selected. This can be achieved by a small change in the react code.
The advantage of this aproach is that the content manager is less likely to make a mistake

Description of alternatives considered

Keep current, which works fine in most simple cases

Screenshots

localhost is the first site alias
image

first alias no longer exists
image

new code in editurl.jsx
image

Affected browser

not relevant, all browsers

@david-poindexter
Copy link
Contributor

Closing per #3751

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants