Skip to content

[stable27] 🐣 Start with implementing bots #10139

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.

Sign up for GitHub

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

Jump to bottom
Merged
nickvergessen merged 21 commits into from
Aug 8, 2023
Merged

[stable27] 🐣 Start with implementing bots #10139

nickvergessen merged 21 commits into from
Aug 8, 2023

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Aug 7, 2023 •
edited
Loading

Backport of #9458 + #10138

Conflicted due to PRs from https://github.com/nextcloud/spreed/pulls?q=is%3Apr+author%3Ashdehnavi

@nickvergessen nickvergessen added 3. to review enhancement feature: chat 💬 Chat and system messages feature: api 🛠️ OCS API for conversations, chats and participants feature: bots 🤖 /commands in chat messages labels Aug 7, 2023
@nickvergessen nickvergessen self-assigned this Aug 7, 2023
@Antreesy Antreesy mentioned this pull request Aug 7, 2023
Closed
nickvergessen and others added 20 commits August 8, 2023 10:46
@nickvergessen
feat(webhooks): Start with implementing webhooks for chat messages
7ae69c8 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
feat(Message): Allow parsing messages without a current participant
fac8f13 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
feat(bots): Allow to read system messages as well
b1a89a8 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix(bots): Set the OCS APIRequest header so Nextclouds can be the Bot…
fa5f671 
… endpoint

Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix(bots): Set timeout to 5s (we wait async if necessary)
38aaa0c 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
feat(bots): Add commands and events to install bots
6b6090e 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix(bots): Use the fixed CLI url
71ee2c8 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
feat(bots): Add API to list, enable and disable bots in a conversation
65893d2 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
ci(psalm): Fix psalm issues and baseline
395cb5c 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
docs: Add occ docs for bot commands
bf86dca 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix(bots): Update name, description and state when reinstalling only
aa32008 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix(bots): Return Bot data when enabling/disabling
ac3d82e 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
tests(integration): Add integration tests for the call summary bot
5a68ea2 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix: Fix code quality based on review
b66fd19 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix: Typo in row attribute names
4ae7a45 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix(chat): Fix actor detection when type and id are provided
49e69db 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
feat(bot): Allow reactions by bots
13c5ced 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix: Block bot commands on incompatible versions
e2de469 
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen
fix(l10n): 27 is not using a non-breakable space
957851e 
Signed-off-by: Joas Schilling <[email protected]>
@Antreesy @nickvergessen
add section in ConversationSettingsDialog.vue for enabling / disablin…
6d90f06 
…g bots

Signed-off-by: Maksim Sukharev <[email protected]>
@nickvergessen nickvergessen force-pushed the backport/9458/stable27 branch 2 times, most recently from 6ef2a30 to 6d90f06 Compare August 8, 2023 14:43
@nickvergessen nickvergessen requested a review from Antreesy August 8, 2023 14:47
@nickvergessen nickvergessen enabled auto-merge August 8, 2023 14:47
Antreesy
Antreesy approved these changes Aug 8, 2023
View reviewed changes
Copy link
Contributor

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested with call summary bot

  • Frontend part is working

@nickvergessen nickvergessen merged commit 4be15ac into stable27 Aug 8, 2023
@nickvergessen nickvergessen deleted the backport/9458/stable27 branch August 8, 2023 16:15
@nickvergessen nickvergessen mentioned this pull request Aug 10, 2023
Merged
@nickvergessen nickvergessen mentioned this pull request Sep 16, 2023
Merged
@Magicrafter13 Magicrafter13 mentioned this pull request Sep 21, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Antreesy Antreesy Antreesy approved these changes

Assignees

@nickvergessen nickvergessen

Labels

3. to review enhancement feature: api 🛠️ OCS API for conversations, chats and participants feature: bots 🤖 /commands in chat messages feature: chat 💬 Chat and system messages

Projects

💬 Talk team
Archived in project

Milestone

v17.1.0-rc.1

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen @Antreesy