Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

feat: implement /status #48

Merged
merged 1 commit into from
Jul 1, 2022
Merged

feat: implement /status #48

merged 1 commit into from
Jul 1, 2022

Conversation

bfabio
Copy link
Member

@bfabio bfabio commented Jul 1, 2022

Fix #9.

LorenzoS92
LorenzoS92 approved these changes Jul 1, 2022
View reviewed changes
Copy link
Contributor

@LorenzoS92 LorenzoS92 left a comment

Choose a reason for hiding this comment

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

Cool!

@bfabio bfabio marked this pull request as ready for review July 1, 2022 12:21
@sebbalex
Copy link
Member

sebbalex commented Jul 1, 2022

nice!
This reflect the actual status of the platform itself. Maybe as a developer I expect to see the entire developers.italia* ecosystem status (this would be veeeery nice :P) wdyt?

@bfabio
Copy link
Member Author

bfabio commented Jul 1, 2022

@sebbalex that's interesting, you mean like adding info to the response with the status of other components? We can track it in an issue 👍

@bfabio bfabio merged commit ea31bc9 into italia:main Jul 1, 2022
@bfabio bfabio deleted the status branch July 1, 2022 13:39
@sebbalex sebbalex mentioned this pull request Jul 1, 2022
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@sebbalex sebbalex sebbalex approved these changes

@LorenzoS92 LorenzoS92 LorenzoS92 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement the status endpoint
3 participants
@bfabio @sebbalex @LorenzoS92