Skip to content

Comments

Extract canSendMessage function#14909

Merged
sampaiodiego merged 7 commits intodevelopfrom
canSendMessage
Jul 2, 2019
Merged

Extract canSendMessage function#14909
sampaiodiego merged 7 commits intodevelopfrom
canSendMessage

Conversation

@ggazzo
Copy link
Member

@ggazzo ggazzo commented Jul 1, 2019

No description provided.

@ggazzo ggazzo requested a review from sampaiodiego July 1, 2019 17:57
import { hasPermissionAsync } from './hasPermission';
import { Subscriptions, Rooms } from '../../../models/server/raw';

const subscriptionProjection = {
Copy link
Member

Choose a reason for hiding this comment

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

please change to:

const subscriptionOptions = {
	projection: {
		blocked: 1,
		blocker: 1,
	},
};

ggazzo and others added 5 commits July 2, 2019 12:11
Co-Authored-By: Diego Sampaio <chinello@gmail.com>
Co-Authored-By: Diego Sampaio <chinello@gmail.com>
Co-Authored-By: Diego Sampaio <chinello@gmail.com>
@ggazzo ggazzo requested a review from d-gubert July 2, 2019 19:14
@sampaiodiego sampaiodiego changed the title [NEW] canSendMessage async Extract canSendMessage function Jul 2, 2019
@sampaiodiego sampaiodiego merged commit 3ce57f4 into develop Jul 2, 2019
@sampaiodiego sampaiodiego deleted the canSendMessage branch July 2, 2019 19:28
@sampaiodiego sampaiodiego mentioned this pull request Aug 2, 2019
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.

2 participants