Skip to content

feat: include IP addresses in system info endpoint#2375

Merged
nicomiguelino merged 1 commit intoScreenly:masterfrom
nicomiguelino:include-ip-addresses-in-info-endpoint
Jun 30, 2025
Merged

feat: include IP addresses in system info endpoint#2375
nicomiguelino merged 1 commit intoScreenly:masterfrom
nicomiguelino:include-ip-addresses-in-info-endpoint

Conversation

@nicomiguelino
Copy link
Contributor

@nicomiguelino nicomiguelino commented Jun 30, 2025

Description

Adds a new ip_addresses field in the response of GET /api/v2/info

Checklist

  • I have performed a self-review of my own code.
  • New and existing unit tests pass locally and on CI with my changes.
  • I have done an end-to-end test for Raspberry Pi devices.
  • I have tested my changes for x86 devices.
  • I added a documentation for the changes I have made (when necessary).

@nicomiguelino nicomiguelino added the api Changes to REST API label Jun 30, 2025
@github-project-automation github-project-automation bot moved this to In progress in Anthias Jun 30, 2025
@nicomiguelino nicomiguelino force-pushed the include-ip-addresses-in-info-endpoint branch from 1ecb79f to 9bf2ba1 Compare June 30, 2025 19:05
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 30, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@nicomiguelino nicomiguelino marked this pull request as ready for review June 30, 2025 19:10
@nicomiguelino nicomiguelino requested a review from a team as a code owner June 30, 2025 19:10
@nicomiguelino nicomiguelino merged commit 34f855e into Screenly:master Jun 30, 2025
8 checks passed
@nicomiguelino nicomiguelino deleted the include-ip-addresses-in-info-endpoint branch June 30, 2025 19:10
@github-project-automation github-project-automation bot moved this from In progress to Done in Anthias Jun 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Changes to REST API

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant