Skip to content

fix(conversation): prevent creating conversations without a name - #7042

Open
mahibi wants to merge 1 commit into
masterfrom
avoidConversationCreationWithoutName
Open

mahibi wants to merge 1 commit into
masterfrom
avoidConversationCreationWithoutName

Conversation

@mahibi

@mahibi mahibi commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator

Disable the create button while the conversation name is blank and ignore creation requests with a blank name in the view model.

Assisted-by: Claude Code:claude-opus-5-5

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Disable the create button while the conversation name is blank and
ignore creation requests with a blank name in the view model.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
@mahibi mahibi added this to the 26.0.0 milestone Nov 12, 2026
@mahibi mahibi self-assigned this Nov 12, 2026
@mahibi mahibi added the 3. to review Waiting for reviews label Nov 12, 2026
@mahibi
mahibi requested a review from rapterjet2004 November 12, 2026 17:48
@github-actions

Copy link
Copy Markdown
Contributor

📱 QA build

Download app-qa-debug.apk
QR code Open the QR code for this download
Commit 6795817
Version 7042
Available until 7 days after this build

The QA build installs alongside a released Nextcloud app, so you can keep
using your existing install while testing.

Downloading the file requires a GitHub account, so open this link on the
device you want to test on, or transfer the APK to it.

@github-actions

Copy link
Copy Markdown
Contributor

Codacy

Lint

TypemasterPR
Warnings1716
Errors00

SpotBugs

CategoryBaseNew
Bad practice77
Correctness88
Dodgy code4040
Internationalization33
Malicious code vulnerability33
Performance77
Security1111
Total7979

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant