Skip to content

Real admin page#130

Draft
KarimJC wants to merge 10 commits intomainfrom
real-admin-page
Draft

Real admin page#130
KarimJC wants to merge 10 commits intomainfrom
real-admin-page

Conversation

@KarimJC
Copy link
Contributor

@KarimJC KarimJC commented Mar 23, 2026

Description

[Link to Ticket](insert the link to your ticket inside the parenthesis here)
Please include a summary of the changes and the related issue. Please also
include relevant motivation, context, and images!

How Has This Been Tested?

Please describe the tests that you manually ran to verify your changes (beyond any unit/integration tests written and ran).

Screenshots

Please provide screenshots of manual testing (Scalar, frontend pages, etc.)

Checklist

General

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas

Backend Changes (if applicable)

  • I have written unit tests for my code
  • I have written integration tests where appropriate
  • New and existing unit tests pass locally with my changes
  • I have run make api-validate and make api-gen to keep the openAPI spec up to date
  • I have tested error handling and edge cases
  • Database migrations are included (if applicable)

Frontend Changes (if applicable)

  • I have tested in multiple browsers (Chrome, Firefox, Safari)
  • Mobile responsive design verified (if applicable)
  • I have run make generate-api to keep the API hooks up to date
  • For mobile features: Tested on both iOS and Android devices
  • For mobile features: Screenshots from both iOS and Android included below
  • No console errors or warnings
  • Loading states and error states handled

Mobile Screenshots (if applicable)

iOS

[Add iOS screenshots here]

Android

[Add Android screenshots here]

@github-actions
Copy link

Backend Test Results

Test Summary ✅

  • Passed: 302
  • Failed: 0
  • Skipped: 0
  • Status: All tests passed!

Coverage ⚠️

  • Total Coverage: 50.9%
  • Status: Coverage is below recommended threshold (70%)

Coverage report and test logs have been uploaded as artifacts.

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.

1 participant