8000 fix(meetings): reword meetigns related messages on Home page and in title by ShGKme · Pull Request #17320 · nextcloud/spreed · GitHub
[go: up one dir, main page]

Skip to content

fix(meetings): reword meetigns related messages on Home page and in title#17320

Open
ShGKme wants to merge 2 commits intomainfrom
fix/app-name-3
Open

fix(meetings): reword meetigns related messages on Home page and in title#17320
ShGKme wants to merge 2 commits intomainfrom
fix/app-name-3

Conversation

@ShGKme
Copy link
Contributor
@ShGKme ShGKme commented Mar 9, 2026

☑️ Resolves

  • Remove app name from the strings
  • Fallback meeting conversation title:
    • Talk conversation for event -> Meeting conversation
  • Reword the placeholder on the Home page

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
Talk conversation for event Meeting conversation
Uploading image.png… image

🏁 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

ShGKme added 2 commits March 9, 2026 10:32
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@ShGKme ShGKme added this to the 🏖️ Next Major (34) milestone Mar 9, 2026
@ShGKme ShGKme self-assigned this Mar 9, 2026
@ShGKme ShGKme added bug feature: meetings 📅 Covering the webinary usecase incl. Lobby labels Mar 9, 2026

// So we can unset names & descriptions in case the user deleted them
$this->roomService->setName($room, $name ?? $this->l10n->t('Talk conversation for event'));
$this->roomService->setName($room, $name ?? $this->l10n->t('Meeting conversation'));
Copy link
Contributor

Choose a reason for hiding this comment

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

We are using 'Meeting' string as a fallback in frontend already, could be reusing that

Suggested change
$this->roomService->setName($room, $name ?? $this->l10n->t('Meeting conversation'));
$this->roomService->setName($room, $name ?? $this->l10n->t('Meeting'));

:title="t('spreed', 'Schedule meetings')"
:subtitle="t('spreed', 'You don\'t have any upcoming meetings')"
:description="t('spreed', 'Schedule a meeting from your calendar. A Talk conversation needs to be set as location to show up here')">
:description="t('spreed', 'Calendar events with a conversation link as the location are shown here')">
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe worth mentioning, that it also covers 'Meetings scheduled from the conversation'

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

Labels

bug feature: meetings 📅 Covering the webinary usecase incl. Lobby

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0