Skip to content

fix(compass-collection): apply project-level setting in schema confirmation screen CLOUDP-333852 #7348

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(compass-collection): apply project-level setting in schema confirmation screen CLOUDP-333852 #7348

wants to merge 1 commit into from
+7 −5

Conversation

kpamaran
Copy link
Collaborator

@kpamaran kpamaran commented Sep 19, 2025 •
edited
Loading

Description

The previous setting should have been the project-level setting instead of the user-level one. The AIOptInModal already uses the project-level setting

Checklist

  • New tests and/or benchmarks are included
  • Documentation is changed or added
  • If this change updates the UI, screenshots/videos are added and a design review is requested
  • I have signed the MongoDB Contributor License Agreement (https://www.mongodb.com/legal/contributor-agreement)

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@kpamaran kpamaran requested a review from a team as a code owner September 19, 2025 18:22
@kpamaran kpamaran requested review from mabaasit, ncarbon and jcobis and removed request for mabaasit September 19, 2025 18:22
@kpamaran kpamaran changed the title Apply project-level setting in schema confirmation screen fix(compass-collection) apply project-level setting in schema confirmation screen CLOUDP-333852 Sep 19, 2025
@kpamaran kpamaran added fix no release notes Fix or feature not for release notes labels Sep 19, 2025
@kpamaran kpamaran changed the title fix(compass-collection) apply project-level setting in schema confirmation screen CLOUDP-333852 fix(compass-collection): apply project-level setting in schema confirmation screen CLOUDP-333852 Sep 19, 2025
jcobis
jcobis approved these changes Sep 19, 2025
View reviewed changes
Copy link
Collaborator

@jcobis jcobis left a comment

Choose a reason for hiding this comment

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

LGTM

@kpamaran kpamaran added the blocked Waiting for something before merging/reviewing further label Sep 19, 2025
@kpamaran
Copy link
Collaborator Author

blocking merge until manual testing can confirm

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

@Anemy Anemy Anemy approved these changes

@jcobis jcobis jcobis approved these changes

@ncarbon ncarbon Awaiting requested review from ncarbon

Assignees
No one assigned
Labels
blocked Waiting for something before merging/reviewing further fix no release notes Fix or feature not for release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kpamaran @Anemy @jcobis