Skip to content

Allow to disable surveys #9349

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.

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

Merged
merged 1 commit into from
Jun 24, 2025
Merged

Allow to disable surveys #9349

merged 1 commit into from
Jun 24, 2025

Conversation

BykhovDenis
Copy link
Member

No description provided.

Signed-off-by: Denis Bykhov <[email protected]>
@BykhovDenis BykhovDenis requested review from haiodo and Copilot June 24, 2025 07:17
Copy link

Connected to Huly®: UBERF-11773

Copy link
Contributor

@Copilot 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

Adds configuration options to disable and describe the survey plugin and updates all locale files with new labels and descriptions.

  • Extended the survey plugin’s string definitions with ConfigLabel and ConfigDescription.
  • Updated each locale JSON to include translations for the new config keys.
  • Modified the global model builder to register the survey model with additional metadata (label, description, enabled, beta, classFilter).

Reviewed Changes

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

Show a summary per file
File Description
plugins/survey/src/index.ts Added ConfigLabel and ConfigDescription properties to strings
plugins/survey-assets/lang/zh.json Added Chinese translations for config label/description
plugins/survey-assets/lang/ru.json Added Russian translations for config label/description
plugins/survey-assets/lang/pt.json Added Portuguese translations for config label/description
plugins/survey-assets/lang/ja.json Added Japanese translations for config label/description
plugins/survey-assets/lang/it.json Added Italian translations for config label/description
plugins/survey-assets/lang/fr.json Added French translations for config label/description
plugins/survey-assets/lang/es.json Added Spanish translations for config label/description
plugins/survey-assets/lang/en.json Added English translations for config label/description
plugins/survey-assets/lang/de.json Added German translations for config label/description
plugins/survey-assets/lang/cs.json Added Czech translations for config label/description
models/all/src/index.ts Imported survey default and registered the model with metadata
Comments suppressed due to low confidence (2)

plugins/survey/src/index.ts:100

  • Consider adding documentation (e.g., README update or docstring) for the new ConfigLabel and ConfigDescription keys to explain their purpose and how they are used in the UI.
    ConfigLabel: '' as IntlString,

models/all/src/index.ts:462

  • Add or update unit/integration tests to verify that the survey model respects the new enabled: false flag and correctly applies the provided label and description metadata.
      {

@BykhovDenis BykhovDenis merged commit 11b3e51 into develop Jun 24, 2025
23 of 24 checks passed
@BykhovDenis BykhovDenis deleted the Allow-to-disable-surveys branch June 24, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants