Skip to content

Commit

Permalink
Improved code quality (#2)
Browse files Browse the repository at this point in the history
* add  "/zoom setting usePMI" command

* update from 'master' branch

* save usePMI as user preference

* fix error handling in /zoom usePMI setting command

* update from upstream

* create meeting without PMI

* finish feature

* fix lint

* tidy go module

* update tests, optimize code

* tidy the go module

* [MI-1996]: Resolved the bug in use PMI feature of zoom plugin
1. Resolved the problem of  500 internal server error which we were getting on clicking create new meeting when our use PMI was set to ask
2. Added c.apiURL in URL
3. Added auto suggestion for use PMI

* [MI-1996]: Resolved merge conflicts

* [MI-1996]: Resolved the error of make: *** No rule to make target 'webapp/node_modules'. Stop

* [MI-1996]: Removed error of check-style

* [MI-1996]: Resolved the error faced in golangci-lint

* [MI-1996]: Review fixes done
1. Added constants at various places

* [MI-1996]:Review fixes done
1. Added EOF
2. Added Proper comments

* [MI-1996]:Resolved the panic that we were getting on running slash command of zoom plugin

* [MI-1996]: Work in Progress

* [MI-1996]:Added properly with and without PMI feature of zoom plugin

* [MI-1996]: Improved code quality

* [MI-1996]: Fixed CI errors

* [MI-1996]:Review fixes done
1.Improved code quality
2.Added various constants

* [MI-1996]: Review fixes done
1. Improved code quality
2. Changed few names

* [MI-1996]:Review fixes done
1. Improved code quality
2. Improved code readability
3. Resolved gramatical mistakes

* [MI-1996]: Review fixes done
1. Changed few names

* [MI-1996]: Review fixes done
1. Improved code readability
2. Removed indirect dependencies
3. Changed few names

* [MI-1996]: Review fixes done
1. Improved code quality
2. Returned json instead of text

* Fixed CI errors

* [MI-1996]:Review fixes done
1. Improved code quality

* [MI-1996]: Improved code quality

Co-authored-by: lantrungseo <[email protected]>
  • Loading branch information
Nityanand13 and andrew-the-drawer authored Nov 14, 2022
1 parent 347f4ab commit 0aadfd9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions server/command.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const (
helpText = `* |/zoom start| - Start a zoom meeting`
oAuthHelpText = `* |/zoom connect| - Connect to Zoom
* |/zoom disconnect| - Disconnect from Zoom`
settingHelpText = `* |/zoom settings| - Configure settings options`
settingHelpText = `* |/zoom settings| - Update your preferences`
alreadyConnectedText = "Already connected"
zoomPreferenceCategory = "plugin:zoom"
zoomPMISettingName = "use-pmi"
Expand Down Expand Up @@ -244,7 +244,7 @@ func (p *Plugin) runDisconnectCommand(user *model.User) (string, error) {

// runHelpCommand runs command to display help text.
func (p *Plugin) runHelpCommand() (string, error) {
text := starterText + strings.ReplaceAll(helpText+settingHelpText, "|", "`")
text := starterText + strings.ReplaceAll(helpText+"\n"+settingHelpText, "|", "`")
if p.configuration.EnableOAuth {
text += "\n" + strings.ReplaceAll(oAuthHelpText, "|", "`")
}
Expand Down

0 comments on commit 0aadfd9

Please sign in to comment.