Skip to content

General data showing#40

Merged
karnishein merged 22 commits intomasterfrom
general-data-showing
Feb 6, 2026
Merged

General data showing#40
karnishein merged 22 commits intomasterfrom
general-data-showing

Conversation

@karnishein
Copy link
Contributor

did some backend for the data, and a table to display it. still not sure the table works, will check after we'll have db and shyte.

image

<Route path="/scout" element={<ScoutMatch />} />
</Routes>
<>
<GeneralDataTable filters={{ "match[type]": "qualification" }} />
Copy link
Contributor

Choose a reason for hiding this comment

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

remove before merging

@karnishein karnishein self-assigned this Feb 5, 2026
@karnishein karnishein added the feature A new Feature! label Feb 5, 2026
throw new Error(`Server Error: ${errorText}`);
}

const data = await response.json();

Check warning

Code scanning / ESLint

Disallow assigning a value with type `any` to variables and properties Warning

Unsafe assignment of an any value.
}

const data = await response.json();
return data.calculatedFuel as TeamNumberAndFuelData;

Check warning

Code scanning / ESLint

Disallow type assertions that narrow a type Warning

Unsafe assertion from any detected: consider using type guards or a safer assertion.
}

const data = await response.json();
return data.calculatedFuel as TeamNumberAndFuelData;

Check warning

Code scanning / ESLint

Disallow member access on a value with type `any` Warning

Unsafe member access .calculatedFuel on an any value.
import { generalCalculateFuel } from "../fuel/fuel-general";
import { StatusCodes } from "http-status-codes";

import type { BPS, FuelObject, GeneralFuelData } from "@repo/scouting_types";

Check warning

Code scanning / ESLint

Disallow unused variables Warning

'BPS' is defined but never used.
@karnishein karnishein merged commit fdca9fc into master Feb 6, 2026
1 check passed
@karnishein karnishein deleted the general-data-showing branch February 6, 2026 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature A new Feature!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants