Skip to content

Add supplier_footprint_mismatch_warning when footprint string does not match with the jlcpcb footprint#2202

Merged
imrishabh18 merged 2 commits intomainfrom
supplier-part-warning
Apr 28, 2026
Merged

Add supplier_footprint_mismatch_warning when footprint string does not match with the jlcpcb footprint#2202
imrishabh18 merged 2 commits intomainfrom
supplier-part-warning

Conversation

@imrishabh18
Copy link
Copy Markdown
Member

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tscircuit-core-benchmarks Ready Ready Preview, Comment Apr 28, 2026 9:01pm

Request Review

@imrishabh18 imrishabh18 changed the title Add supplier_footprint_mismatch_warning when footprint string does not match with the jlcpcb footprint Add supplier_footprint_mismatch_warning when footprint string does not match with the jlcpcb footprint Apr 28, 2026
doInitialSupplierFootprintMismatchWarning(): void {
NormalComponent_doInitialSupplierFootprintMismatchWarning(
this,
(name, effect) => this._queueAsyncEffect(name, effect),
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.

no second arguments for these methods

"digikey",
"mouser",
"lcsc",
]
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.

can you import this from props? Is this the best place to define this?

Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

see comment

@imrishabh18 imrishabh18 merged commit 8b485cd into main Apr 28, 2026
10 checks passed
@imrishabh18 imrishabh18 deleted the supplier-part-warning branch April 28, 2026 21:05
@tscircuitbot
Copy link
Copy Markdown
Contributor


Thank you for your contribution! 🎉

PR Rating: ⭐⭐
Impact: Minor

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


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.

3 participants