Skip to content

feat(save-user-data): extend user data abstract class for api backend COMPASS-9558 #7114

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

Open
wants to merge 37 commits into
base: main
Choose a base branch
from

Conversation

myang1220
Copy link
Contributor

@myang1220 myang1220 commented Jul 15, 2025

Description

Extend the IUserData abstract class to create an AtlasUserData class that uses the endpoints created in CCS. Contains unit testing.

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

See comments

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 Jul 15, 2025
@myang1220 myang1220 added the no release notes Fix or feature not for release notes label Jul 15, 2025
}

return true;
} catch (error) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

How does the logging look? Are there any changes that I should make to make this more consistent with Compass behavior?

@myang1220 myang1220 requested review from Anemy and gribnoysup July 18, 2025 21:21
@myang1220 myang1220 marked this pull request as ready for review July 18, 2025 21:21
@Copilot Copilot AI review requested due to automatic review settings July 18, 2025 21:21
@myang1220 myang1220 requested a review from a team as a code owner July 18, 2025 21:21
Copy link

@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

This PR extends the user data abstract class system by implementing a new AtlasUserData class that communicates with Atlas backend API endpoints for cloud-based user data storage, as an alternative to the existing file-based storage.

  • Adds AtlasUserData class implementing the IUserData interface for API-based storage
  • Includes comprehensive test coverage for all CRUD operations and URL construction
  • Updates exports to make the new class available to consumers

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.

File Description
packages/compass-user-data/src/user-data.ts Implements AtlasUserData class with HTTP-based CRUD operations
packages/compass-user-data/src/user-data.spec.ts Adds comprehensive test suite for AtlasUserData functionality
packages/compass-user-data/src/index.ts Exports the new AtlasUserData class

myang1220 and others added 3 commits July 18, 2025 17:25
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Merge branch 'extend-user-data' of https://github.com/mongodb-js/compass into extend-user-data
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.

3 participants