kredits/hubot-kredits
9
0
Fork 0
Code Issues 8 Pull Requests Releases 9 Activity

Zoom integration #53

Merged
bumi merged 11 commits from feature/zoom into master 2020-04-16 15:34:39 +00:00
Conversation 11 Commits 11 Files Changed 2 +2 -2
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c4ef8de018 - Show all commits

4
integrations/zoom.js
View File

@@ -61,13 +61,13 @@ module.exports = async function(robot, kredits) {
raucao commented 2020-04-16 09:21:06 +00:00 (Migrated from github.com)
Review
Copy Link
  const { Contributor, Contribution } = kredits;
```suggestion const { Contributor, Contribution } = kredits; ```
raucao commented 2020-04-16 09:21:06 +00:00 (Migrated from github.com)
Review
Copy Link
  const { Contributor, Contribution } = kredits;
```suggestion const { Contributor, Contribution } = kredits; ```
raucao commented 2020-04-16 09:23:48 +00:00 (Migrated from github.com)
Review
Copy Link
        const contribution = {
```suggestion const contribution = { ```
raucao commented 2020-04-16 09:23:48 +00:00 (Migrated from github.com)
Review
Copy Link
        const contribution = {
```suggestion const contribution = { ```
raucao commented 2020-04-16 09:26:43 +00:00 (Migrated from github.com)
Review
Copy Link

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".
raucao commented 2020-04-16 09:26:43 +00:00 (Migrated from github.com)
Review
Copy Link

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".
raucao commented 2020-04-16 09:37:06 +00:00 (Migrated from github.com)
Review
Copy Link
      robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`);
```suggestion robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`); ```
raucao commented 2020-04-16 09:37:06 +00:00 (Migrated from github.com)
Review
Copy Link
      robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`);
```suggestion robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`); ```
raucao commented 2020-04-16 09:38:17 +00:00 (Migrated from github.com)
Review
Copy Link
          robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`);
```suggestion robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`); ```
raucao commented 2020-04-16 09:38:17 +00:00 (Migrated from github.com)
Review
Copy Link
          robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`);
```suggestion robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`); ```
raucao commented 2020-04-16 09:39:58 +00:00 (Migrated from github.com)
Review
Copy Link
  robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => {
```suggestion robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => { ```
raucao commented 2020-04-16 09:39:58 +00:00 (Migrated from github.com)
Review
Copy Link
  robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => {
```suggestion robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => { ```
const participants = await getMeetingParticipants(data.uuid);
if (meetingDetails.duration < 15 || meetingDetails.participants_count < 3) {
robot.logger.info(`[hubot-kredits] ignoring meeting: uuid:${data.uuid} duration:${meetingDetails.duration} participants_count:${meetingDetails.participants_count}`);
raucao commented 2020-04-16 09:21:06 +00:00 (Migrated from github.com)
Review
Copy Link
  const { Contributor, Contribution } = kredits;
```suggestion const { Contributor, Contribution } = kredits; ```
raucao commented 2020-04-16 09:23:48 +00:00 (Migrated from github.com)
Review
Copy Link
        const contribution = {
```suggestion const contribution = { ```
raucao commented 2020-04-16 09:26:43 +00:00 (Migrated from github.com)
Review
Copy Link

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".
raucao commented 2020-04-16 09:37:06 +00:00 (Migrated from github.com)
Review
Copy Link
      robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`);
```suggestion robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`); ```
raucao commented 2020-04-16 09:38:17 +00:00 (Migrated from github.com)
Review
Copy Link
          robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`);
```suggestion robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`); ```
raucao commented 2020-04-16 09:39:58 +00:00 (Migrated from github.com)
Review
Copy Link
  robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => {
```suggestion robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => { ```
robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`);
raucao commented 2020-04-16 09:21:06 +00:00 (Migrated from github.com)
Review
Copy Link
  const { Contributor, Contribution } = kredits;
```suggestion const { Contributor, Contribution } = kredits; ```
raucao commented 2020-04-16 09:23:48 +00:00 (Migrated from github.com)
Review
Copy Link
        const contribution = {
```suggestion const contribution = { ```
raucao commented 2020-04-16 09:26:43 +00:00 (Migrated from github.com)
Review
Copy Link

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".
raucao commented 2020-04-16 09:37:06 +00:00 (Migrated from github.com)
Review
Copy Link
      robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`);
```suggestion robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`); ```
raucao commented 2020-04-16 09:38:17 +00:00 (Migrated from github.com)
Review
Copy Link
          robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`);
```suggestion robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`); ```
raucao commented 2020-04-16 09:39:58 +00:00 (Migrated from github.com)
Review
Copy Link
  robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => {
```suggestion robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => { ```
return;
}
participants.forEach(p => {
createContributionFor(p, meetingDetails)
.then(tx => {
robot.logger.info(`[hubot-kredits] contribution created: ${tx.hash}`);
raucao commented 2020-04-16 09:21:06 +00:00 (Migrated from github.com)
Review
Copy Link
  const { Contributor, Contribution } = kredits;
```suggestion const { Contributor, Contribution } = kredits; ```
raucao commented 2020-04-16 09:23:48 +00:00 (Migrated from github.com)
Review
Copy Link
        const contribution = {
```suggestion const contribution = { ```
raucao commented 2020-04-16 09:26:43 +00:00 (Migrated from github.com)
Review
Copy Link

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".
raucao commented 2020-04-16 09:37:06 +00:00 (Migrated from github.com)
Review
Copy Link
      robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`);
```suggestion robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`); ```
raucao commented 2020-04-16 09:38:17 +00:00 (Migrated from github.com)
Review
Copy Link
          robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`);
```suggestion robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`); ```
raucao commented 2020-04-16 09:39:58 +00:00 (Migrated from github.com)
Review
Copy Link
  robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => {
```suggestion robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => { ```
robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`);
raucao commented 2020-04-16 09:21:06 +00:00 (Migrated from github.com)
Review
Copy Link
  const { Contributor, Contribution } = kredits;
```suggestion const { Contributor, Contribution } = kredits; ```
raucao commented 2020-04-16 09:23:48 +00:00 (Migrated from github.com)
Review
Copy Link
        const contribution = {
```suggestion const contribution = { ```
raucao commented 2020-04-16 09:26:43 +00:00 (Migrated from github.com)
Review
Copy Link

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".
raucao commented 2020-04-16 09:37:06 +00:00 (Migrated from github.com)
Review
Copy Link
      robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`);
```suggestion robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`); ```
raucao commented 2020-04-16 09:38:17 +00:00 (Migrated from github.com)
Review
Copy Link
          robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`);
```suggestion robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`); ```
raucao commented 2020-04-16 09:39:58 +00:00 (Migrated from github.com)
Review
Copy Link
  robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => {
```suggestion robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => { ```
})
});
}
raucao commented 2020-04-16 09:21:06 +00:00 (Migrated from github.com)
Review
Copy Link
  const { Contributor, Contribution } = kredits;
```suggestion const { Contributor, Contribution } = kredits; ```
raucao commented 2020-04-16 09:21:06 +00:00 (Migrated from github.com)
Review
Copy Link
  const { Contributor, Contribution } = kredits;
```suggestion const { Contributor, Contribution } = kredits; ```
raucao commented 2020-04-16 09:23:48 +00:00 (Migrated from github.com)
Review
Copy Link
        const contribution = {
```suggestion const contribution = { ```
raucao commented 2020-04-16 09:23:48 +00:00 (Migrated from github.com)
Review
Copy Link
        const contribution = {
```suggestion const contribution = { ```
raucao commented 2020-04-16 09:26:43 +00:00 (Migrated from github.com)
Review
Copy Link

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".
raucao commented 2020-04-16 09:26:43 +00:00 (Migrated from github.com)
Review
Copy Link

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".

This is a bit of an issue, because we sometimes use the Weekly Call room for other meetings. Not sure what the best solution is. Maybe just something generic like "Team/Community Call".
raucao commented 2020-04-16 09:37:06 +00:00 (Migrated from github.com)
Review
Copy Link
      robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`);
```suggestion robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`); ```
raucao commented 2020-04-16 09:37:06 +00:00 (Migrated from github.com)
Review
Copy Link
      robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`);
```suggestion robot.logger.info(`[hubot-kredits] Ignoring zoom call ${data.uuid} (duration: ${meetingDetails.duration}, participants_count: ${meetingDetails.participants_count})`); ```
raucao commented 2020-04-16 09:38:17 +00:00 (Migrated from github.com)
Review
Copy Link
          robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`);
```suggestion robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`); ```
raucao commented 2020-04-16 09:38:17 +00:00 (Migrated from github.com)
Review
Copy Link
          robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`);
```suggestion robot.logger.info(`[hubot-kredits] Contribution created: ${tx.hash}`); ```
raucao commented 2020-04-16 09:39:58 +00:00 (Migrated from github.com)
Review
Copy Link
  robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => {
```suggestion robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => { ```
raucao commented 2020-04-16 09:39:58 +00:00 (Migrated from github.com)
Review
Copy Link
  robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => {
```suggestion robot.router.post('/incoming/kredits/zoom/'+process.env.KREDITS_WEBHOOK_TOKEN, (req, res) => { ```