Skip to content

Commit

Permalink
Clean-up old functionality
Browse files Browse the repository at this point in the history
  • Loading branch information
jwilander committed Sep 18, 2019
1 parent f1a1c52 commit f69989e
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 117 deletions.
35 changes: 6 additions & 29 deletions server/command.go
Original file line number Diff line number Diff line change
Expand Up @@ -59,35 +59,12 @@ func (p *Plugin) executeCommand(c *plugin.Context, args *model.CommandArgs) (str
return fmt.Sprintf("We could not get channel members (channelId: %v)", args.ChannelId), nil
}

meetingID := 0
personal := false

// Determine if the user is sending command in DM or channel
channel, tmpErr := p.API.GetChannel(args.ChannelId)
if tmpErr != nil {
return fmt.Sprintf("We could not get channel members (channelId: %v)", args.ChannelId), nil
}

if channel.Type == "D" {
// create a personal zoom meeting
personal = true
ru, clientErr := p.zoomClient.GetUser(user.Email)
if clientErr != nil {
return "We could not verify your Mattermost account in Zoom. Please ensure that your Mattermost email address matches your Zoom login email address.", nil
}
meetingID = ru.Pmi
} else {
// create a channel zoom meeting
meeting := &zoom.Meeting{
Type: zoom.MeetingTypeInstant,
}

rm, clientErr := p.zoomClient.CreateMeeting(meeting, user.Email)
if clientErr != nil {
return "We could not create and start a meeting in Zoom. Please ensure that your Mattermost email address matches your Zoom login email address.", nil
}
meetingID = rm.ID
// create a personal zoom meeting
ru, clientErr := p.zoomClient.GetUser(user.Email)
if clientErr != nil {
return "We could not verify your Mattermost account in Zoom. Please ensure that your Mattermost email address matches your Zoom login email address.", nil
}
meetingID := ru.Pmi

zoomURL := strings.TrimSpace(config.ZoomURL)
if len(zoomURL) == 0 {
Expand All @@ -105,7 +82,7 @@ func (p *Plugin) executeCommand(c *plugin.Context, args *model.CommandArgs) (str
"meeting_id": meetingID,
"meeting_link": meetingURL,
"meeting_status": zoom.WebhookStatusStarted,
"meeting_personal": personal,
"meeting_personal": true,
},
}

Expand Down
33 changes: 6 additions & 27 deletions server/http.go
Original file line number Diff line number Diff line change
Expand Up @@ -134,33 +134,12 @@ func (p *Plugin) handleStartMeeting(w http.ResponseWriter, r *http.Request) {
return
}

meetingID := req.MeetingID
personal := req.Personal

if meetingID == 0 && req.Personal {
ru, clientErr := p.zoomClient.GetUser(user.Email)
if clientErr != nil {
http.Error(w, clientErr.Error(), clientErr.StatusCode)
return
}
meetingID = ru.Pmi
}

if meetingID == 0 {
personal = false

meeting := &zoom.Meeting{
Type: zoom.MeetingTypeInstant,
Topic: req.Topic,
}

rm, clientErr := p.zoomClient.CreateMeeting(meeting, user.Email)
if clientErr != nil {
http.Error(w, clientErr.Error(), clientErr.StatusCode)
return
}
meetingID = rm.ID
ru, clientErr := p.zoomClient.GetUser(user.Email)
if clientErr != nil {
http.Error(w, clientErr.Error(), clientErr.StatusCode)
return
}
meetingID := ru.Pmi

zoomURL := strings.TrimSpace(config.ZoomURL)
if len(zoomURL) == 0 {
Expand All @@ -178,7 +157,7 @@ func (p *Plugin) handleStartMeeting(w http.ResponseWriter, r *http.Request) {
"meeting_id": meetingID,
"meeting_link": meetingURL,
"meeting_status": zoom.WebhookStatusStarted,
"meeting_personal": personal,
"meeting_personal": true,
"meeting_topic": req.Topic,
},
}
Expand Down
61 changes: 0 additions & 61 deletions server/zoom/meeting.go

This file was deleted.

0 comments on commit f69989e

Please sign in to comment.