Skip to content

✨ feat: allow copy flags between projects#817

Merged
deleteLater merged 7 commits intomainfrom
feat/copy-flags-cross-projects
Dec 10, 2025
Merged

✨ feat: allow copy flags between projects#817
deleteLater merged 7 commits intomainfrom
feat/copy-flags-cross-projects

Conversation

@deleteLater
Copy link
Copy Markdown
Contributor

@deleteLater deleteLater commented Dec 8, 2025

copy flags across projects

image

audit log comment

image

Copy link
Copy Markdown
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 enables cross-project feature flag copying by extending the existing copy functionality to work across different projects, not just within the same project. The feature now displays all available environments from all accessible projects in a "ProjectName/EnvName" format, and tracks the source environment in audit logs for better traceability.

  • Display current source environment and allow selection from all projects' environments
  • Add source environment tracking to audit logs with "Copied from" comment
  • Implement RN.IsInScope method for resource hierarchy validation in cross-project scenarios

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
modules/front-end/src/app/core/components/copy-feature-flag-modal/copy-feature-flag-modal.component.ts Refactored to load all projects asynchronously and build environment options with project prefixes; added source environment tracking
modules/front-end/src/app/core/components/copy-feature-flag-modal/copy-feature-flag-modal.component.html Added current environment display section and loading state for environment selector
modules/front-end/src/app/core/components/copy-feature-flag-modal/copy-feature-flag-modal.component.less Added styling for current environment display and increased selector width to accommodate longer labels
modules/front-end/src/styles/nz-select.less Added loading icon margin styling
modules/front-end/src/locale/messages.xlf Updated line number references for localization strings
modules/front-end/src/locale/messages.zh.xlf Updated line number references for localization strings and translations
modules/back-end/src/Application/FeatureFlags/CopyToEnv.cs Added SourceEnvId property, implemented source environment name resolution, and included source in audit log comments
modules/back-end/src/Api/Controllers/FeatureFlagController.cs Updated CopyToEnvAsync to accept and pass source environment ID
modules/back-end/tests/Domain.UnitTests/Resources/RNTests.cs Added comprehensive tests for RN.IsInScope method to validate resource hierarchy logic

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

deleteLater and others added 3 commits December 8, 2025 18:10
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@cosmos-explorer cosmos-explorer left a comment

Choose a reason for hiding this comment

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

LGTM

@deleteLater deleteLater changed the title feat: allow copy flags between projects ✨ feat: allow copy flags between projects Dec 10, 2025
@deleteLater deleteLater merged commit 490bee1 into main Dec 10, 2025
2 checks passed
@deleteLater deleteLater deleted the feat/copy-flags-cross-projects branch December 10, 2025 08:12
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.

3 participants