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

[IMPROVE] Avoid unnecessary calls to Meteor.user() on client #11212

Merged
merged 7 commits into from
Oct 18, 2018

Conversation

ggazzo
Copy link
Member

@ggazzo ggazzo commented Jun 21, 2018

Closes #ISSUE_NUMBER

@ggazzo ggazzo changed the title [WIP] avoiding [WIP] avoiding Meteor.user() on client Jun 21, 2018
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11212 June 21, 2018 13:46 Inactive
@ggazzo ggazzo requested a review from sampaiodiego June 21, 2018 14:28
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-11212 June 21, 2018 14:47 Inactive
@ggazzo ggazzo added this to the 0.67.0 milestone Jul 3, 2018
@ggazzo ggazzo changed the title [WIP] avoiding Meteor.user() on client [IMPROVE] avoiding Meteor.user() on client Jul 5, 2018
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-11212 July 19, 2018 14:46 Inactive
@theorenck theorenck modified the milestones: 0.67.0, 0.68.0 Jul 19, 2018
@Hudell Hudell self-requested a review July 19, 2018 19:40
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-11212 August 16, 2018 20:34 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11212 October 2, 2018 14:52 Inactive
@RocketChat RocketChat deleted a comment Oct 2, 2018
@RocketChat RocketChat deleted a comment Oct 2, 2018
@RocketChat RocketChat deleted a comment Oct 2, 2018
@RocketChat RocketChat deleted a comment Oct 2, 2018
@sampaiodiego sampaiodiego changed the title [IMPROVE] avoiding Meteor.user() on client [IMPROVE] Avoid unnecessary calls to Meteor.user() on client Oct 18, 2018
@sampaiodiego sampaiodiego merged commit 3243634 into develop Oct 18, 2018
@sampaiodiego sampaiodiego deleted the avoid-meteor-user branch October 18, 2018 11:49
@sampaiodiego sampaiodiego mentioned this pull request Oct 27, 2018
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.

5 participants