Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Communities v2 prototype: DM copy updates - #5153

Merged
turt2live merged 6 commits into
developfrom
travis/copy2
Sep 2, 2020
Merged

Communities v2 prototype: DM copy updates#5153
turt2live merged 6 commits into
developfrom
travis/copy2

Conversation

@turt2live

@turt2live turt2live commented Aug 28, 2020

Copy link
Copy Markdown
Member

Prototype behaviour:

  • If looking at the DM dialog while in a community, try to communicate that it doesn't necessarily affect the community.
    • DMs don't add people the community
    • Scary external people might be included in suggestions
    • Rationale for copy changes: de-scaring the user.

image
image

@turt2live turt2live changed the title Minor copy addition to DM dialog for communities prototype Communities v2 prototype: DM copy updates Aug 28, 2020
@turt2live
turt2live marked this pull request as ready for review August 28, 2020 19:23
@turt2live
turt2live requested a review from a team August 28, 2020 19:29

if (kind === 'suggestions' && TagOrderStore.getSelectedPrototypeTag()) {
const summary = GroupStore.getSummary(TagOrderStore.getSelectedPrototypeTag());
const communityName = summary?.profile?.name || TagOrderStore.getSelectedPrototypeTag();

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ah cool, didn't realise we can use this ?. syntax now, yay! 😍

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

tbh I completely forgot about it up until writing this 😀

It showed up as supported when we started supporting typescript

@turt2live
turt2live merged commit c849f56 into develop Sep 2, 2020
@turt2live
turt2live deleted the travis/copy2 branch September 2, 2020 16:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants