-
Notifications
You must be signed in to change notification settings - Fork 4
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
WIP: Join Command #4
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not a clean up.
commands/DebugCommand.ts
Outdated
import { BlockBuilder, TextObjectType } from "@rocket.chat/apps-engine/definition/uikit"; | ||
import { IUser } from "@rocket.chat/apps-engine/definition/users"; | ||
|
||
// I know i can use the app logger but i like this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this please. This is a serious project, we don't want to commit any staging resources here.
commands/HelpCommand.ts
Outdated
import { IUser } from "@rocket.chat/apps-engine/definition/users"; | ||
|
||
export class HelpCommand implements ISlashCommand { | ||
public command = "helpmeet"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A slashcommand for showing help message? Why?
import { IUser } from "@rocket.chat/apps-engine/definition/users"; | ||
import { getMeetingUrl } from "../functions/getMeetingUrl"; | ||
|
||
export class JoinCommand implements ISlashCommand { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not being used.
commands/Meet.ts
Outdated
await modify.getScheduler().scheduleRecurring({ | ||
id: `${commandroom.slugifiedName}`, | ||
// interval: `${mins} ${hrs} * * ${dayind}`, | ||
interval: '20 seconds', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is your test code. Neither appropriate to commit nor part of the scope of this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
neither is this part of the pr scope.
@@ -0,0 +1,33 @@ | |||
import { IProcessor } from "@rocket.chat/apps-engine/definition/scheduler"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not part of join command.
@@ -0,0 +1,23 @@ | |||
import { IHttp, IModify, IPersistence, IRead } from "@rocket.chat/apps-engine/definition/accessors" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not part of join command
commands/StopReminder.ts
Outdated
@@ -0,0 +1,58 @@ | |||
// import { IRead, IModify, IHttp, IPersistence, IModifyCreator, IMessageBuilder } from "@rocket.chat/apps-engine/definition/accessors"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this file.
@@ -0,0 +1,83 @@ | |||
import { IRead, IModify, IHttp, IPersistence, IModifyCreator, IMessageBuilder, ILogger } from "@rocket.chat/apps-engine/definition/accessors"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not in PR scope.
import { multipleProcessors } from './reminder/processors/multipleProcessors'; | ||
import { weeklyNotification } from './reminder/processors/weeklyNotification'; | ||
import { AppSettings } from './settings/appsettings'; | ||
import { BbbSettings } from './settings/bbbsettings'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to clean up the imports.
BbbMeetApp.ts
Outdated
await configuration.slashCommands.provideSlashCommand(new ScheduleMeetCommand()) | ||
await configuration.slashCommands.provideSlashCommand(new MeetCommand()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are not registering the command that this PR is about.
The app settings will have a setting which will be a string of room names which are supposed to be the meeting channels where the join command will work. When the join command is called: