Skip to content

Handle numeric company IDs in cache refresh#35

Merged
MDCYT merged 1 commit intomainfrom
dev
Mar 3, 2026
Merged

Handle numeric company IDs in cache refresh#35
MDCYT merged 1 commit intomainfrom
dev

Conversation

@MDCYT
Copy link
Copy Markdown
Owner

@MDCYT MDCYT commented Mar 3, 2026

Support responses where company entries are numeric IDs by returning the number directly in the mapping step; update related comments. Changes applied to src/routes/v2/peruserver/trucky/top-km/index.js and monthly.js. The subsequent filter still ensures only finite numeric IDs are kept, allowing the cache refresh to handle varied API response formats.

Support responses where company entries are numeric IDs by returning the number directly in the mapping step; update related comments. Changes applied to src/routes/v2/peruserver/trucky/top-km/index.js and monthly.js. The subsequent filter still ensures only finite numeric IDs are kept, allowing the cache refresh to handle varied API response formats.
@MDCYT MDCYT merged commit 8c68f8b into main Mar 3, 2026
1 of 2 checks passed
@deepsource-io
Copy link
Copy Markdown
Contributor

deepsource-io bot commented Mar 3, 2026

DeepSource Code Review

We reviewed changes in 95af6f9...51dfd31 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Mar 3, 2026 12:48a.m. Review ↗

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.

1 participant