Skip to content
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

refactor(db-proxy): refactor accessor constructor in database-proxy, add objectid type as global in runtime; #1738

Merged
merged 6 commits into from
Dec 14, 2023

Conversation

maslow
Copy link
Member

@maslow maslow commented Dec 14, 2023

No description provided.

Copy link

sweep-ai bot commented Dec 14, 2023

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

Copy link

💥 laf nodejs runtime init image built successfully for pr 1738

ghcr.io/labring/runtime-node-init:pr-1738
docker.io/lafyun/runtime-node-init:pr-1738

Copy link

💥 laf nodejs runtime image built successfully for pr 1738

ghcr.io/labring/runtime-node:pr-1738
docker.io/lafyun/runtime-node:pr-1738

Copy link

💥 laf server image built successfully for pr 1738

ghcr.io/labring/laf-server:pr-1738
docker.io/lafyun/laf-server:pr-1738

Copy link

💥 laf web image built successfully for pr 1738

ghcr.io/labring/laf-web:pr-1738
docker.io/lafyun/laf-web:pr-1738

@maslow maslow changed the title Refact db proxy constructor refactor(db-proxy): refactor accessor constructor in database-proxy, add objectid type as global in runtime; Dec 14, 2023
@maslow maslow merged commit 3ee12db into main Dec 14, 2023
8 checks passed
@maslow maslow deleted the refact-db-proxy-constructor branch December 14, 2023 07:19
maslow added a commit that referenced this pull request Dec 19, 2023
…add objectid type as global in runtime; (#1738)

* refactor(db-proxy): refact constructor of accessor
* add ObjectId to global in runtime
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant