Skip to content

Conversation

@AlexAndBear
Copy link
Contributor

@AlexAndBear AlexAndBear commented Jul 18, 2025

Description

Related Issue

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Types of changes

  • Bugfix
  • Enhancement (a change that doesn't break existing code or deployments)
  • Breaking change (a modification that affects current functionality)
  • Technical debt (addressing code that needs refactoring or improvements)
  • Tests (adding or improving tests)
  • Documentation (updates or additions to documentation)
  • Maintenance (like dependency updates or tooling adjustments)

@AlexAndBear AlexAndBear requested a review from tammi-23 July 18, 2025 07:13
@AlexAndBear AlexAndBear self-assigned this Jul 18, 2025
Copilot AI review requested due to automatic review settings July 18, 2025 07:13
@github-project-automation github-project-automation bot moved this to Qualification in OpenCloud Team Board Jul 18, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR standardizes quote marks used in modal titles across the application by replacing straight double quotes (") with angle quotes (»«) for consistency. The changes focus on user-facing messages in modal dialogs and notifications.

Key Changes:

  • Updates quote formatting in space rename modal titles to use angle quotes with proper i18n interpolation
  • Standardizes success and error message titles in SaveAs modal to use consistent angle quote formatting

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
useSpaceActionsRename.ts Updates rename space modal title to use angle quotes and proper i18n interpolation pattern
SaveAsModal.vue Standardizes quote marks in both success and error message titles to use angle quotes

Copy link
Contributor

@tammi-23 tammi-23 left a comment

Choose a reason for hiding this comment

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

👍 Reviewed and tested. Looks very good.

@tammi-23 tammi-23 merged commit ce56831 into main Jul 18, 2025
23 checks passed
@tammi-23 tammi-23 deleted the issues/966 branch July 18, 2025 12:01
@github-project-automation github-project-automation bot moved this from Qualification to Done in OpenCloud Team Board Jul 18, 2025
This was referenced Jul 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Renaming a Space don´t show Quotes like in the US #911

3 participants