Skip to content

fix: export for dto added, required when only API is run#342

Merged
andrasfuchs merged 1 commit intofreerouting:masterfrom
imrishabh18:fix/export-for-dto
Sep 10, 2024
Merged

fix: export for dto added, required when only API is run#342
andrasfuchs merged 1 commit intofreerouting:masterfrom
imrishabh18:fix/export-for-dto

Conversation

@imrishabh18
Copy link
Contributor

@imrishabh18 imrishabh18 commented Sep 10, 2024

Description

When I wanted to run only the API, and then hitting the status http://localhost:37864/api/v1/system/status endpoint. I was receiving this export issue
Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make field public java.lang.String app.freerouting.api.dto.SystemStatus.status accessible: module app.freerouting does not "exports app.freerouting.api.dto" to module com.google.gson

This change fixes it

Checklist

  • Code compiles correctly
  • Created tests which fail without the change (if possible)
  • All tests passing
  • Extended the README / documentation, if necessary

@andrasfuchs
Copy link
Collaborator

andrasfuchs commented Sep 10, 2024

Great, thank you @imrishabh18!

Please keep in mind that the API is very experimental at this stage, and there is no documentation available.
Having said that feel free to test it, and let me know what you think!

@andrasfuchs andrasfuchs merged commit 610bb34 into freerouting:master Sep 10, 2024
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