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.

Sign up for GitHub

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

Jump to bottom

[App] Same message to be broadcast to multiple group #660

Open
danlg opened this issue Oct 18, 2016 · 0 comments
Open

[App] Same message to be broadcast to multiple group #660

danlg opened this issue Oct 18, 2016 · 0 comments
Assignees
@amngupta
Labels
newfeature

Comments

@danlg
Copy link
Owner

danlg commented Oct 18, 2016 •
edited
Loading

Necessary to implement #648 and fix #657

Smartix.Messages.Schema = new SimpleSchema({
    group: { type: String, optional:true },
    groups: { type: [String], optional:true }

the new code will only insert messages with field groups populated but will be able to read from field group for backward capability reason
amngupta added a commit that referenced this issue Oct 20, 2016
@amngupta
[App] Same message to be broadcast to multiple group #660
fd19dc5 
- Stash changes before working on bottleneck issue
amngupta added a commit that referenced this issue Oct 21, 2016
@amngupta
[App] Same message to be broadcast to multiple group #660
f333757 
- Test code need to complete implementation
amngupta added a commit that referenced this issue Oct 23, 2016
@amngupta
[App] Same message to be broadcast to multiple group #660
bf59c6b 
- Removed unused helpers getGroupName
- Regression on getGroupName in list view due to [Groups]
- Fixed full-calendar in case of entries with other addons
amngupta added a commit that referenced this issue Oct 23, 2016
@amngupta
[App] Same message to be broadcast to multiple group #660
af9d6c8 
- Added new method broadcastNewsMessage to broadcast news to multiple groups
- Fixed relevant publish and methods
amngupta added a commit that referenced this issue Oct 25, 2016
@amngupta
[App] Same message to be broadcast to multiple group #660
b8dcd5b 
- Added case for groups
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amngupta amngupta

Labels
newfeature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@danlg @amngupta