Skip to content

feat(api): Custom prices on industry cost API#917

Draft
kennethjor wants to merge 9 commits into
mainfrom
801-custom-material-prices-on-industry-cost-api
Draft

feat(api): Custom prices on industry cost API#917
kennethjor wants to merge 9 commits into
mainfrom
801-custom-material-prices-on-industry-cost-api

Conversation

@kennethjor

Copy link
Copy Markdown
Contributor

No description provided.

@kennethjor kennethjor linked an issue Jun 24, 2025 that may be closed by this pull request
@netlify

netlify Bot commented Jun 24, 2025

Copy link
Copy Markdown

Deploy Preview for everef-docs canceled.

Name Link
🔨 Latest commit 79f473c
🔍 Latest deploy log https://app.netlify.com/projects/everef-docs/deploys/685b6f49278cac0008909ec2

@qlty-cloud-legacy

qlty-cloud-legacy Bot commented Jun 24, 2025

Copy link
Copy Markdown

Code Climate has analyzed commit 79f473c and detected 0 issues on this pull request.

View more on Code Climate.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 24, 2025

Copy link
Copy Markdown

Deploying eve-ref with  Cloudflare Pages  Cloudflare Pages

Latest commit: 79f473c
Status:🚫  Build failed.

View logs

@kennethjor

Copy link
Copy Markdown
Contributor Author

For java-native, I had to implement a custom method for serialising to deepObject. typescript-fetch insists on using deep object notation even for the inputs. typescript-axios uses some sort of dot-notation like ?input.product_id=645 which, as far as I can tell, isn't even in the OpenAPI spec. It also appears none of the generators URL-encode the brackets [] for deep object notation, which Helidon doesn't like. I might have to abandon deep object completely because of this.

Alternatively, I could just encode the type IDs directly. Since none of the other values are raw numbers, they'll be easy to detect. This would be encoded simply as &34=4.32.

Need to try out https://openapi-ts.dev/openapi-fetch/

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Custom material prices on industry cost API

1 participant