Skip to content

Add dynamodb apl#449

Merged
lkostrowski merged 2 commits intomainfrom
dynamo-apl
Jul 28, 2025
Merged

Add dynamodb apl#449
lkostrowski merged 2 commits intomainfrom
dynamo-apl

Conversation

@lkostrowski
Copy link
Copy Markdown
Member

No description provided.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jul 28, 2025

🦋 Changeset detected

Latest commit: c59a26a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@saleor/app-sdk Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 28, 2025

Codecov Report

❌ Patch coverage is 98.70130% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.79%. Comparing base (2d5ef93) to head (c59a26a).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/APL/dynamodb/memory-apl-repository.ts 92.59% 2 Missing ⚠️
src/APL/dynamodb/index.ts 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #449      +/-   ##
==========================================
+ Coverage   83.87%   84.79%   +0.91%     
==========================================
  Files         107      114       +7     
  Lines        3511     3742     +231     
  Branches      591      636      +45     
==========================================
+ Hits         2945     3173     +228     
- Misses        555      557       +2     
- Partials       11       12       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Copy Markdown
Contributor

Released snapshot build with @dev tag in npm with version: 0.0.0-pr-20250728123253.

Install it with:

pnpm add @saleor/app-sdk@0.0.0-pr-20250728123253

@lkostrowski lkostrowski marked this pull request as ready for review July 28, 2025 12:39

private env: Envs;

static create(deps: { env: Envs; table: UsedTable }) {
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.

Suggestion: maybe we can write it docs that create it to create new repo under the hood and constructor is to reuse existing e.g from app?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yup, I'm adding docs right now

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@lkostrowski lkostrowski merged commit 2fc132c into main Jul 28, 2025
11 checks passed
@lkostrowski lkostrowski deleted the dynamo-apl branch July 28, 2025 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants