Skip to content

Conversation

HoonBaek
Copy link
Contributor

@HoonBaek HoonBaek commented Feb 1, 2023

For Internal Contributors

UIKIT-2766

Description Of Changes

  • Add string set

    • CHANNEL_SETTING__OPERATORS__ADD_BUTTON: 'Add'
    • CHANNEL_SETTING__MODERATION__EMPTY_BAN: 'No banned members yet'
    • CHANNEL_SETTING__MODERATION__ALL_BAN: 'All banned members'
  • Apply string set

    • on the BannedUserList component
      • No banned members yet
      • All banned members
      • Unban on the context menu of the banned user list
    • on the BannedUsersModal component
      • Unban on the context menu of the banned user list
    • on the AddOperatorsModal component
      • Add for the submit button
    • on the InviteUsersModal component
      • Invite for the submit button
    • on the MutedMembersModal component
      • Unmute on the context menu of the muted user list
  • Remove string set

    • CHANNEL_SETTING__UNMUTE: 'Unmute'
  • Replace the string set to CHANNEL_SETTING__MODERATION__UNMUTE on the MutedMemberList component

  • Modify CSS

    • To improve the word break with StringSet customization on the submit button

Types Of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply_

  • Bugfix
  • New feature
  • Documentation (correction or otherwise)
  • Cosmetics (whitespace, appearance (ex) Prettier)
  • Build configuration
  • Improvement (refactor code)

@HoonBaek HoonBaek requested a review from sravan-s February 1, 2023 05:12
@HoonBaek HoonBaek self-assigned this Feb 1, 2023
>
<>
{stringSet.CHANNEL_SETTING__UNMUTE}
{stringSet.CHANNEL_SETTING__MODERATION__UNMUTE}
Copy link
Contributor

Choose a reason for hiding this comment

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

This might be breaking chnage

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
{stringSet.CHANNEL_SETTING__MODERATION__UNMUTE}
{stringSet?.CHANNEL_SETTING__UNMUTE || stringSet.CHANNEL_SETTING__MODERATION__UNMUTE}

@HoonBaek HoonBaek merged commit d208f74 into main Feb 1, 2023
@HoonBaek HoonBaek deleted the fix/UIKIT-2766/Missing-string-set-in-channel-settings branch February 1, 2023 11:03
HoonBaek added a commit that referenced this pull request Feb 2, 2023
* Add string set
  * CHANNEL_SETTING__OPERATORS__ADD_BUTTON: 'Add'
  * CHANNEL_SETTING__MODERATION__EMPTY_BAN: 'No banned members yet'
  * CHANNEL_SETTING__MODERATION__ALL_BAN: 'All banned members'
* Apply string set
  * on the BannedUserList component
    * `No banned members yet`
    * `All banned members`
    * `Unban` on the context menu of the banned user list
  * on the BannedUsersModal component
    * `Unban` on the context menu of the banned user list
  * on the AddOperatorsModal component
    * `Add` for the submit button
  * on the InviteUsersModal component
    * `Invite` for the submit button
  * on the MutedMembersModal component
    * `Unmute` on the context menu of the muted user list

* Remove string set
  * CHANNEL_SETTING__UNMUTE: 'Unmute'
* Replace the string set to CHANNEL_SETTING__MODERATION__UNMUTE on the MutedMemberList component

* Modify CSS
  * To improve the word break with StringSet customization on the submit button
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants