Skip to content

Conversation

@mabaasit
Copy link
Collaborator

Description

Checklist

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)

@github-actions github-actions bot added the feat label Oct 28, 2024
@mabaasit mabaasit added the no release notes Fix or feature not for release notes label Oct 28, 2024
workspaces.openCollectionWorkspace(connectionId, namespace, {
newTab: true,
initialSubtab:
connectionInfo && connectionSupports(connectionInfo, 'globalWrites')
Copy link
Collaborator

@paula-stacho paula-stacho Oct 28, 2024

Choose a reason for hiding this comment

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

connectionSupports won't be sufficient. we'll need to filter out the admin/config/local collections and also views. adding @djechlin-mongodb, we'll need to sync this with the hiding of the tab
more context: https://mongodb.slack.com/archives/C03DU5ATTB6/p1729869659313749

Copy link
Collaborator

@paula-stacho paula-stacho Oct 28, 2024

Choose a reason for hiding this comment

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

sorry for only realising this now 🙈

Copy link
Collaborator

Choose a reason for hiding this comment

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

You can't create views through this modal and I'm pretty sure that even db users with admin roles can't create collections inside special-ish namespaces (but for anything that is defined as special-ish by mongodb-ns package we probably can add a filter here, yeah)

Copy link
Collaborator

Choose a reason for hiding this comment

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

ah right, thanks @gribnoysup !

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

added in 0d124af

Copy link
Collaborator

@paula-stacho paula-stacho left a comment

Choose a reason for hiding this comment

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

lgtm

@mabaasit mabaasit merged commit 8554a6a into main Oct 29, 2024
29 checks passed
@mabaasit mabaasit deleted the COMPASS-8280-open-global-writes-tab branch October 29, 2024 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat no release notes Fix or feature not for release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants