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

Remove dependency of RC namespace in rc-sandstorm #13227

Merged
merged 20 commits into from
Jan 23, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
6f749bc
Move integrations models to rc-models
MarcosSpessatto Jan 21, 2019
5dec8ce
Move composeMessage function to rc-utils
MarcosSpessatto Jan 21, 2019
583cb87
Move PushNotifications class to push-notifications package
MarcosSpessatto Jan 21, 2019
039e7cd
Import variables to remove dependency of RC namespace
MarcosSpessatto Jan 21, 2019
d49540b
Import variables to remove RC namespace dependency inside rc-lib/serv…
MarcosSpessatto Jan 22, 2019
4cede2e
Import variables to remove RC namespace inside rc-lib/server/methods
MarcosSpessatto Jan 22, 2019
5ecc7b8
Import variables to remove dependency with RC namespace
MarcosSpessatto Jan 22, 2019
f3280a6
Remove duplicated server settings located in rc-lib
MarcosSpessatto Jan 22, 2019
f526680
Merge remote-tracking branch 'origin/depackaging' into globals/rc-lib…
rodrigok Jan 22, 2019
8d4c70f
Merge branch 'globals/rc-lib-wave-5' into globals/rc-lib-wave-6
MarcosSpessatto Jan 22, 2019
fb7666d
Merge branch 'globals/rc-lib-wave-6' into globals/lib-settings
MarcosSpessatto Jan 22, 2019
f457877
Export function and lib function of rc-lib but keep it inside RC name…
MarcosSpessatto Jan 22, 2019
3b6ffa6
Import variables from functions and lib to remove RC namespace depend…
MarcosSpessatto Jan 22, 2019
27043cb
Merge remote-tracking branch 'origin/depackaging' into globals/lib-fu…
rodrigok Jan 22, 2019
6ab7f28
Remove dependency of RC namespace in rc-sandstorm
MarcosSpessatto Jan 22, 2019
f00a322
Merge branch 'globals/lib-functions' into globals/lib-functions-import
MarcosSpessatto Jan 22, 2019
6043a3d
Merge branch 'globals/lib-functions-import' into globals/lib-x-sandstorm
MarcosSpessatto Jan 22, 2019
36c17f2
Merge remote-tracking branch 'origin/depackaging' into globals/lib-fu…
rodrigok Jan 23, 2019
08d8bd1
Merge remote-tracking branch 'origin/globals/lib-functions-import' in…
rodrigok Jan 23, 2019
06a780f
Merge remote-tracking branch 'origin/depackaging' into globals/lib-x-…
rodrigok Jan 23, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions packages/rocketchat-lib/client/lib/sandstorm.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
import { Sandstorm } from 'meteor/rocketchat:sandstorm';

RocketChat.Sandstorm = Sandstorm;
3 changes: 3 additions & 0 deletions packages/rocketchat-lib/package.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ Package.onUse(function(api) {
api.use('rocketchat:logger');
api.use('rocketchat:mailer');
api.use('rocketchat:settings');
api.use('rocketchat:sandstorm');
api.use('rocketchat:authorization');
api.use('mizzao:timesync');
api.use('rocketchat:custom-oauth');
Expand Down Expand Up @@ -146,6 +147,7 @@ Package.onUse(function(api) {
api.addFiles('server/lib/validateEmailDomain.js', 'server');
api.addFiles('server/lib/passwordPolicy.js', 'server');
api.addFiles('server/lib/migrations.js', 'server');
api.addFiles('server/lib/sandstorm.js', 'server');

// SERVER MODELS
api.addFiles('server/models/index.js', 'server');
Expand Down Expand Up @@ -238,6 +240,7 @@ Package.onUse(function(api) {
api.addFiles('client/lib/tooltip.js', 'client');
api.addFiles('client/lib/EmojiPicker.js', 'client');
api.addFiles('client/lib/ui-buttons.js', 'client');
api.addFiles('client/lib/sandstorm.js', 'client');

// CLIENT LIB STARTUP
api.addFiles('client/lib/startup/commands.js', 'client');
Expand Down
3 changes: 3 additions & 0 deletions packages/rocketchat-lib/server/lib/sandstorm.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
import { Sandstorm } from 'meteor/rocketchat:sandstorm';

RocketChat.Sandstorm = Sandstorm;
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import { settings } from 'meteor/rocketchat:settings';
import { callbacks } from 'meteor/rocketchat:callbacks';
import { Subscriptions } from 'meteor/rocketchat:models';
import { roomTypes } from 'meteor/rocketchat:utils';
import { Sandstorm } from 'meteor/rocketchat:sandstorm';
import { callJoinRoom, messageContainsHighlight, parseMessageTextPerUser, replaceMentionedUsernamesWithFullNames } from '../functions/notifications/';
import { sendEmail, shouldNotifyEmail } from '../functions/notifications/email';
import { sendSinglePush, shouldNotifyMobile } from '../functions/notifications/mobile';
Expand Down Expand Up @@ -136,7 +137,7 @@ const sendNotification = async({
}

if (notificationSent) {
RocketChat.Sandstorm.notify(message, [subscription.u._id], `@${ sender.username }: ${ message.msg }`, room.t === 'p' ? 'privateMessage' : 'message');
Sandstorm.notify(message, [subscription.u._id], `@${ sender.username }: ${ message.msg }`, room.t === 'p' ? 'privateMessage' : 'message');
}
};

Expand Down
6 changes: 5 additions & 1 deletion packages/rocketchat-sandstorm/client/index.js
Original file line number Diff line number Diff line change
@@ -1,2 +1,6 @@
import './powerboxListener';
import { Sandstorm } from './powerboxListener';
import './setPath';

export {
Sandstorm,
};
7 changes: 3 additions & 4 deletions packages/rocketchat-sandstorm/client/powerboxListener.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import { Meteor } from 'meteor/meteor';
import { RocketChat } from 'meteor/rocketchat:lib';

RocketChat.Sandstorm = RocketChat.Sandstorm || {};
export const Sandstorm = {};

RocketChat.Sandstorm.request = function() {};
Sandstorm.request = function() {};
if (Meteor.settings.public.sandstorm) {
const callbackMap = {};

Expand All @@ -20,7 +19,7 @@ if (Meteor.settings.public.sandstorm) {
uiView: 'EAZQAQEAABEBF1EEAQH_5-Jn6pjXtNsAAAA',
};

RocketChat.Sandstorm.request = function(interfaceName, cb) {
Sandstorm.request = function(interfaceName, cb) {
const rpcId = Math.random().toString();
callbackMap[rpcId] = cb;
window.parent.postMessage({ powerboxRequest: {
Expand Down
1 change: 0 additions & 1 deletion packages/rocketchat-sandstorm/package.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ Package.describe({
Package.onUse(function(api) {
api.use([
'ecmascript',
'rocketchat:lib',
'jalik:ufs',
'kadira:flow-router',
]);
Expand Down
6 changes: 3 additions & 3 deletions packages/rocketchat-sandstorm/server/events.js
Original file line number Diff line number Diff line change
@@ -1,17 +1,17 @@
import { Meteor } from 'meteor/meteor';
import { RocketChat } from 'meteor/rocketchat:lib';
import { getHttpBridge, waitPromise } from './lib';
import { Sandstorm } from './lib';
import _ from 'underscore';

RocketChat.Sandstorm.notify = function() {};
Sandstorm.notify = function() {};

if (process.env.SANDSTORM === '1') {
const ACTIVITY_TYPES = {
message: 0,
privateMessage: 1,
};

RocketChat.Sandstorm.notify = function(message, userIds, caption, type) {
Sandstorm.notify = function(message, userIds, caption, type) {
const sessionId = message.sandstormSessionId;
if (!sessionId) {
return;
Expand Down
6 changes: 5 additions & 1 deletion packages/rocketchat-sandstorm/server/index.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
import './lib';
import { Sandstorm } from './lib';
import './events';
import './powerbox';

export {
Sandstorm,
};
3 changes: 1 addition & 2 deletions packages/rocketchat-sandstorm/server/lib.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import Future from 'fibers/future';
import { RocketChat } from 'meteor/rocketchat:lib';
import { UploadFS } from 'meteor/jalik:ufs';

RocketChat.Sandstorm = {};
export const Sandstorm = {};

export let getHttpBridge;
export let waitPromise;
Expand Down
8 changes: 4 additions & 4 deletions packages/rocketchat-sandstorm/server/powerbox.js
Original file line number Diff line number Diff line change
@@ -1,15 +1,15 @@
import { Meteor } from 'meteor/meteor';
import { RocketChat } from 'meteor/rocketchat:lib';
import { Sandstorm } from './lib';
import { getHttpBridge, waitPromise } from './lib';

RocketChat.Sandstorm.offerUiView = function() {};
Sandstorm.offerUiView = function() {};

if (process.env.SANDSTORM === '1') {
const Capnp = require('capnp');
const Powerbox = Capnp.importSystem('sandstorm/powerbox.capnp');
const Grain = Capnp.importSystem('sandstorm/grain.capnp');

RocketChat.Sandstorm.offerUiView = function(token, serializedDescriptor, sessionId) {
Sandstorm.offerUiView = function(token, serializedDescriptor, sessionId) {
const httpBridge = getHttpBridge();
const session = httpBridge.getSessionContext(sessionId).context;
const { api } = httpBridge.getSandstormApi(sessionId);
Expand Down Expand Up @@ -43,7 +43,7 @@ if (process.env.SANDSTORM === '1') {
};
},
sandstormOffer(token, serializedDescriptor) {
RocketChat.Sandstorm.offerUiView(token, serializedDescriptor,
Sandstorm.offerUiView(token, serializedDescriptor,
this.connection.sandstormSessionId());
},
});
Expand Down