Skip to content

feat : Update API pricing to use version 6.1#39

Merged
yanuaraditia merged 6 commits intomainfrom
adif/patch-change-shipping-api-version
Apr 7, 2025
Merged

feat : Update API pricing to use version 6.1#39
yanuaraditia merged 6 commits intomainfrom
adif/patch-change-shipping-api-version

Conversation

@muhadif
Copy link
Copy Markdown
Contributor

@muhadif muhadif commented Mar 28, 2025

Updated the shipping price and pickup request endpoints to align with the latest API version 6.1. This ensures compatibility with recent changes and improved functionality.

muhadif added 5 commits March 28, 2025 15:16
Updated the shipping price and pickup request endpoints to align with the latest API version 6.1. This ensures compatibility with recent changes and improved functionality.
Replaced array-based package handling with `RequestPickupDataList` for better management and type safety. Introduced default values in `PackageData` and updated `ShippingRepository` to process packages as arrays before API requests. These changes improve code clarity and maintainability.
Replaced direct array assignment with a method call to add the package to the pickup object. This improves clarity and ensures better alignment with object handling practices.
Previously, the 'note' property defaulted to null, which may have required additional null checks. This change ensures consistency by initializing it with an empty string, simplifying its usage.
Previously, the 'note' property defaulted to null, which may have required additional null checks. This change ensures consistency by initializing it with an empty string, simplifying its usage.
Copy link
Copy Markdown
Collaborator

@yanuaraditia yanuaraditia left a comment

Choose a reason for hiding this comment

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

LGTM

@yanuaraditia yanuaraditia merged commit beb8370 into main Apr 7, 2025
1 check passed
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