Skip to content

Capitalize letters#14566

Merged
turt2live merged 2 commits intoelement-hq:developfrom
TheEvilSkeleton:develop
Jul 23, 2020
Merged

Capitalize letters#14566
turt2live merged 2 commits intoelement-hq:developfrom
TheEvilSkeleton:develop

Conversation

@TheEvilSkeleton
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

@nadonomy should be done now 3e37f21

@nadonomy nadonomy self-requested a review July 21, 2020 11:10
Copy link
Copy Markdown
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