Skip to content

Feature/8030 as a developer i want to have a new endpoint in data access service to return latest available date of wave buoys so that imos live project can have it#115

Merged
utas-raymondng merged 5 commits intomainfrom
feature/8030-as-a-developer-i-want-to-have-a-new-endpoint-in-data-access-service-to-return-latest-available-date-of-wave-buoys-so-that-imos-live-project-can-have-it
Feb 5, 2026
Merged

Feature/8030 as a developer i want to have a new endpoint in data access service to return latest available date of wave buoys so that imos live project can have it#115
utas-raymondng merged 5 commits intomainfrom
feature/8030-as-a-developer-i-want-to-have-a-new-endpoint-in-data-access-service-to-return-latest-available-date-of-wave-buoys-so-that-imos-live-project-can-have-it

Conversation

@leslieduan
Copy link
Contributor

No description provided.

Copy link
Contributor

@utas-raymondng utas-raymondng left a comment

Choose a reason for hiding this comment

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

Just format change?

@leslieduan
Copy link
Contributor Author

Just format change?

Yes, before this, Test / pre-commit-and-pytest complained.

Copy link
Contributor

@utas-raymondng utas-raymondng left a comment

Choose a reason for hiding this comment

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

Do you have test case to verify your code?

@leslieduan
Copy link
Contributor Author

Do you have test case to verify your code?

No, I did not. The error was due to pre-commit format check I think.

Copy link
Contributor

@utas-raymondng utas-raymondng left a comment

Choose a reason for hiding this comment

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

LGTM

@utas-raymondng utas-raymondng merged commit 91aa0ea into main Feb 5, 2026
2 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.

2 participants