Skip to content

fix: migrate chat summary feature to NcAssistant* components #15773

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
Antreesy merged 3 commits into from
Sep 2, 2025
Merged

fix: migrate chat summary feature to NcAssistant* components #15773

Antreesy merged 3 commits into from
Sep 2, 2025

Conversation

@Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Sep 1, 2025 •
edited
Loading

☑️ Resolves

  • Bump nextcloud/vue library
  • Fix NcAssistant related interface

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

☀️ Light theme 🌑 Dark Theme
image image
2025-09-01_18h07_30 image

Screencasts:
2025-09-01_17h25_12
2025-09-01_18h07_09

🚧 Tasks

  • ...

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

@Antreesy Antreesy added this to the 🪺 Next RC (32) milestone Sep 1, 2025
@Antreesy Antreesy self-assigned this Sep 1, 2025
@Antreesy Antreesy changed the title chore(deps): bump @nextcloud/vue to v9.0.0-rc.8 fix: migrate chat summary feature to NcAssistant* components Sep 1, 2025
Antreesy
Antreesy commented Sep 1, 2025
View reviewed changes
DorraJaouad
DorraJaouad approved these changes Sep 2, 2025
View reviewed changes
Copy link
Contributor

@DorraJaouad DorraJaouad left a comment

Choose a reason for hiding this comment

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

Code-wise

@Antreesy Antreesy merged commit c0ada12 into main Sep 2, 2025
55 checks passed
@Antreesy Antreesy deleted the fix/noid/assistant branch September 2, 2025 09:24
@nickvergessen nickvergessen mentioned this pull request Sep 4, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

@marcoambrosini marcoambrosini marcoambrosini approved these changes

@nimishavijay nimishavijay nimishavijay approved these changes

@DorraJaouad DorraJaouad DorraJaouad approved these changes

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

Assignees

@Antreesy Antreesy

Labels

3. to review design feature: frontend 🖌️ "Web UI" client

Projects

None yet

Milestone

v22.0.0-rc.1

Development

Successfully merging this pull request may close these issues.

6 participants

@Antreesy @AndyScherzinger @marcoambrosini @nimishavijay @DorraJaouad