Skip to content

Construction cart improvements - #471

Merged
jplacht merged 3 commits into
PRUNplanner:mainfrom
lumivient:improve-construction-cart
Sep 26, 2026
Merged

jplacht merged 3 commits into
PRUNplanner:mainfrom
lumivient:improve-construction-cart

Conversation

@lumivient

Copy link
Copy Markdown
Contributor

Material table footer (cost/weight/volume) and XIT button use Need column, instead of just duplicating the Construction Cart table.

Also fixed PInputNumber bug where +/- buttons were enabled for empty values, causing NaN and broken calculations.

Untitled

@netlify

netlify Bot commented Jun 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for prunplanner-preview ready!

Name Link
🔨 Latest commit 5008623
🔍 Latest deploy log https://app.netlify.com/projects/prunplanner-preview/deploys/6ab772457285860008f06564
😎 Deploy Preview https://deploy-preview-471--prunplanner-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 duplication

Metric Results
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

…ction cart

The backend drops null values (exclude_none), so ships without a name,
sites without PlanetName and buildings without repair/reclaim materials
arrive without those keys and failed zod validation. The construction
cart awaited the storage query without a catch, so any failure left the
cart blank.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jplacht
jplacht merged commit 0438b97 into PRUNplanner:main Sep 26, 2026
9 of 10 checks passed
@lumivient
lumivient deleted the improve-construction-cart branch September 27, 2026 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants