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

Invalid auto created Site Url #3115

Closed
1 task done
Timo-Breumelhof opened this issue Jun 27, 2019 · 6 comments
Closed
1 task done

Invalid auto created Site Url #3115

Timo-Breumelhof opened this issue Jun 27, 2019 · 6 comments
Assignees
Milestone

Comments

@Timo-Breumelhof
Copy link
Contributor

Description of bug

When creating a site of the type "Directory" (child-portal) the generated Site Url can contain spaces, which is not valid.

Steps to reproduce

List the steps to reproduce the behavior:
1 . Create a new site.
2. Enter the Site Title: "My website"
3. Select the ( ) Directory radio
4. DNN generates "dnn9dev.local/My Website" as the "site Url"
Which is incorrect, it should be "dnn9dev.local/My-Website" IMO.
If you don't notice this and click [Create Site] you get an error
It would be good if DNN would replace invalid characters when auto creating this Url

Current result

"dnn9dev.local/My Website

Expected result

"dnn9dev.local/My-Website"

  • 9.3.2
@Timo-Breumelhof
Copy link
Contributor Author

Related: dnnsoftware/Dnn.AdminExperience#1063 ?

@valadas
Copy link
Contributor

valadas commented Jul 29, 2019

I cannot reproduce in 9.4.0 RC maybe this somehow got fixed by something else. @Timo-Breumelhof. The 9.4.0 RC is available as a pre-release, so not meant for production, if you want to give it a try and repport back. Or if you notice it is still an issue after the official 9.4.0 release, please open a new issue.

@valadas valadas closed this as completed Jul 29, 2019
@Timo-Breumelhof
Copy link
Contributor Author

@valadas I just tested this on the RC and it's still an issue.

image

@valadas
Copy link
Contributor

valadas commented Aug 1, 2019

I stand corrected or it relates with some other conditions, today I can reproduce

@valadas valadas reopened this Aug 1, 2019
@valadas
Copy link
Contributor

valadas commented Aug 1, 2019

I am 99% sure this is a UI issue and moving it to AdminExperience...

@valadas valadas transferred this issue from dnnsoftware/Dnn.Platform Aug 1, 2019
@donker donker self-assigned this Sep 29, 2019
@valadas valadas transferred this issue from dnnsoftware/Dnn.AdminExperience Oct 5, 2019
@valadas valadas added this to the 9.4.2 milestone Oct 5, 2019
@valadas valadas modified the milestones: 9.4.2, 9.4.3 Nov 5, 2019
@valadas
Copy link
Contributor

valadas commented Nov 20, 2019

merged

@valadas valadas closed this as completed Nov 20, 2019
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

3 participants