Skip to content

add status check for devices#22

Merged
JooNiv merged 2 commits intorewrite/mainfrom
ft-add-qc-status
Jun 18, 2025
Merged

add status check for devices#22
JooNiv merged 2 commits intorewrite/mainfrom
ft-add-qc-status

Conversation

@Modupef
Copy link
Copy Markdown

@Modupef Modupef commented Jun 18, 2025

Changes:

  • add health check for the devices

@Modupef Modupef requested a review from JooNiv June 18, 2025 11:05
Comment thread src/components/ServiceStatus.jsx Outdated
@@ -1,10 +1,13 @@
import React from 'react'
import { useMemo } from 'react';
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

useMemo seems to be unused

Comment thread src/hooks/useStatus.jsx Outdated
try {
const resp = await fetch (url);
const result = await resp.json();
const device_status = result.data
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

unused variable

@JooNiv JooNiv merged commit 79a7ac2 into rewrite/main Jun 18, 2025
1 of 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