Skip to content

Preselect projects using selected environment - #2013

Open
Stella Huang (StellaHuang95) wants to merge 2 commits into
microsoft:mainfrom
StellaHuang95:stellahuang-microsoft-issue-489-set-as-project-environment-should-pre-se-1ef0d6
Open

Stella Huang (StellaHuang95) wants to merge 2 commits into
microsoft:mainfrom
StellaHuang95:stellahuang-microsoft-issue-489-set-as-project-environment-should-pre-se-1ef0d6

Conversation

@StellaHuang95

Copy link
Copy Markdown
Contributor

Summary

  • preselect projects currently using the chosen environment
  • apply only the selection delta, resetting deselected projects without changing unrelated assignments
  • support the same workflow for a single project and compare manager-specific environment identities safely

Validation

  • npm run lint
  • npm run compile-tests
  • npm run unittest (3,543 passing, 9 pending)
  • npm run compile
  • manually verified cancellation, preselection, set/reset behavior, unrelated-project preservation, and persistence across reloads

Fixes #489

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@rchiodo

Rich Chiodo (rchiodo) commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

🔒 Automated review in progress — Rich Chiodo (@rchiodo) is auto-reviewing this PR.

@StellaHuang95 Stella Huang (StellaHuang95) added the feature-request Request for new features or functionality label Nov 12, 2026
@rchiodo

Rich Chiodo (rchiodo) commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Reviewed commit: 68bf8fea6a0650bccfe5cece84a55cded0d0414d

Result: 🔴 could-not-verify

Verification details

Verification: Automated verification is best effort. The PR author remains responsible for testing.

Summary: [unavailable] Container verification could not start and local execution was not authorized for this PR HEAD: Podman machine &#x27;pyrx-automation&#x27; is not running.

Test runs: none recorded.

@rchiodo Rich Chiodo (rchiodo) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved via Review Center.

@rchiodo Rich Chiodo (rchiodo) added the review-auto:approved Automated review: no blocking findings (approval posted). label Nov 12, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

@bschnurr Bill Schnurr (bschnurr) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved via Review Center.

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

Labels

feature-request Request for new features or functionality review-auto:approved Automated review: no blocking findings (approval posted).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Environment Manager -> Set as Project Environment should pre-select the environments it's already used in

3 participants