Skip to content

Brings location field to v2.x#300

Merged
soareswallace merged 1 commit intov2.xfrom
v2.15.0
Sep 19, 2025
Merged

Brings location field to v2.x#300
soareswallace merged 1 commit intov2.xfrom
v2.15.0

Conversation

@soareswallace
Copy link
Contributor

@soareswallace soareswallace commented Sep 19, 2025

Proposed changes

Brings location field to the v2.x

Checklist

  • Style check and tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@soareswallace soareswallace self-assigned this Sep 19, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces location field support to the v2.x API by adding a Location class and integrating it into login registration functionality.

  • Adds a new Location class with latitude, longitude, and collectedAt fields
  • Integrates location field into RegisterLoginRequest and PostTransactionRequestBody
  • Updates test coverage to include location field usage

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/main/java/com/incognia/common/Location.java New Location class with builder pattern for location data
src/main/java/com/incognia/transaction/login/RegisterLoginRequest.java Adds location field to login request model
src/main/java/com/incognia/transaction/PostTransactionRequestBody.java Adds location field to transaction request body
src/main/java/com/incognia/api/IncogniaAPI.java Maps location field from request to request body
src/test/java/com/incognia/api/IncogniaAPITest.java Updates test to include location field usage

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

* Adding location API Field to the lib

* Upgrade action/cache to v4
@soareswallace soareswallace merged commit 972cd28 into v2.x Sep 19, 2025
4 of 5 checks 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.

3 participants