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

Replaced 'Piwik' with 'Matomo' in console output when creating extra Custom Dimensions #16475

Merged
merged 1 commit into from Sep 26, 2020
Merged

Replaced 'Piwik' with 'Matomo' in console output when creating extra Custom Dimensions #16475

merged 1 commit into from Sep 26, 2020

Conversation

nina-py
Copy link
Contributor

@nina-py nina-py commented Sep 26, 2020

While adding extra custom dimensions to reproduce #16155, I noticed that the output from the console command still says "Your Piwik..." instead of "Your Matomo...". Updated the copy both in the output and in the corresponding tests.

@nina-py
Copy link
Contributor Author

nina-py commented Sep 26, 2020

@tsteur, just a quick question - for a straightforward fix like this that is not directly related to the issue I was working on, is a pull request enough or should I open an issue as well?

@tsteur
Copy link
Member

tsteur commented Sep 26, 2020

Cheers again @nina-py A separate issue is not needed 👍

@tsteur tsteur added the Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc. label Sep 26, 2020
@tsteur tsteur merged commit a49ed63 into matomo-org:4.x-dev Sep 26, 2020
@tsteur tsteur added Bug For errors / faults / flaws / inconsistencies etc. and removed Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc. labels Sep 26, 2020
@nina-py nina-py deleted the piwik-in-customer-facing-code branch September 26, 2020 04:33
@nina-py
Copy link
Contributor Author

nina-py commented Sep 26, 2020

Thanks @tsteur!

@mattab mattab added this to the 4.0.0 milestone Sep 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants