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

Capitalize letters #14566

Merged
merged 2 commits into from
Jul 23, 2020
Merged

Capitalize letters #14566

merged 2 commits into from
Jul 23, 2020

Conversation

TheEvilSkeleton
Copy link
Contributor

No description provided.

@TheEvilSkeleton TheEvilSkeleton changed the title Capitalize and fix [matrix] Capitalize and fix "[matrix]" Jul 17, 2020
@TheEvilSkeleton TheEvilSkeleton changed the title Capitalize and fix "[matrix]" Capitalize letters and fix "[matrix]" Jul 17, 2020
@turt2live turt2live requested a review from a team July 17, 2020 03:56
Copy link
Contributor

@nadonomy nadonomy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @TheEvilSkeleton thanks for this PR!

Unfortunately we aren't able to merge this as is. Although the logomark & logotype for Matrix includes square brackets, we don't include them when calling Matrix in text/by name.

If you're able to update the string to be 'Powered by Matrix' we can merge.

@TheEvilSkeleton TheEvilSkeleton changed the title Capitalize letters and fix "[matrix]" Capitalize letters Jul 20, 2020
@TheEvilSkeleton
Copy link
Contributor Author

@nadonomy should be done now 3e37f21

@nadonomy nadonomy self-requested a review July 21, 2020 11:10
Copy link
Contributor

@nadonomy nadonomy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @TheEvilSkeleton LGTM!

Approving from my side, also paging @vector-im/riot-web for code review & to confirm this meets contribution guidelines.

@nadonomy nadonomy requested a review from a team July 21, 2020 11:11
@turt2live turt2live merged commit 3e37f21 into element-hq:develop Jul 23, 2020
refi64 added a commit to refi64/riot-web-lc that referenced this pull request Jul 28, 2020
[Full Changelog](element-hq/element-web@v1.7.1...v1.7.2)
* Upgrade to React SDK 3.0.0 and JS SDK 8.0.0
* Update from Weblate
[\element-hq#14778](element-hq#14778)
* Capitalize letters
[\element-hq#14566](element-hq#14566)
* Configure eslint package and fix lint issues
[\element-hq#14673](element-hq#14673)
* Riot → Element
[\element-hq#14581](element-hq#14581)
* Remove labs info for the new room list
[\element-hq#14603](element-hq#14603)
* Convince Webpack to use development on CI
[\element-hq#14593](element-hq#14593)
* Move dev dep to the right place
[\element-hq#14572](element-hq#14572)
* Bump lodash from 4.17.15 to 4.17.19
[\element-hq#14552](element-hq#14552)
* Update all mobile links to match marketing site
[\element-hq#14541](element-hq#14541)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants