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

Jump to bottom

Dashboard title with space #14820

Closed
Laul0 opened this issue Aug 27, 2019 · 1 comment · Fixed by #14869
Closed

Dashboard title with space #14820

Laul0 opened this issue Aug 27, 2019 · 1 comment · Fixed by #14869
Assignees
@katebutler
Labels
Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Milestone
3.12.0

Comments

@Laul0
Copy link

Laul0 commented Aug 27, 2019

Summary

It is possible to create several custom Dashboard without any restriction into the title of them.

Current behavior

When I create a new Dashboard with a space into the title, the space is encoded (replaced by a %20)

Screen Shot 2019-08-19 at 19 15 39

Expected behavior

Several possibilities are available

  1. Prevent user to create a Dashboard with some special chars (useless because the dashboard title is not used into the URL)
  2. Allow special char but display them without encodage
@tsteur tsteur added Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should. labels Aug 27, 2019
@tsteur tsteur added this to the 3.12.0 milestone Aug 27, 2019
@tsteur
Copy link
Member

tsteur commented Aug 27, 2019

Thanks @Laul0 I can reproduce that 👍

@katebutler katebutler self-assigned this Sep 4, 2019
@katebutler katebutler mentioned this issue Sep 8, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@katebutler katebutler

Labels
Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Milestone
3.12.0
Development

Successfully merging a pull request may close this issue.

Stop urlencoding dashboard names before sending to API matomo-org/matomo
3 participants
@tsteur @katebutler @Laul0