Skip to content

Feat/registration form styling#31

Open
heitzuli wants to merge 12 commits intomainfrom
feat/registration-form-styling
Open

Feat/registration form styling#31
heitzuli wants to merge 12 commits intomainfrom
feat/registration-form-styling

Conversation

@heitzuli
Copy link
Copy Markdown
Collaborator

The styling of the registration form.

@hollannikas hollannikas mentioned this pull request Mar 29, 2025
@heitzuli heitzuli requested review from Kitsupitsu and Copilot March 29, 2025 17:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request introduces new interfaces for organization-related models and adds API functions and a React component for displaying organization data, which contributes toward the registration form styling. Key changes include:

  • Defining TypeScript interfaces for person, facility, time range, and organization.
  • Creating a new Organizations component to render a table of organizations.
  • Adding API functions for posting registration data and fetching organizations.

Reviewed Changes

Copilot reviewed 5 out of 8 changed files in this pull request and generated 1 comment.

File Description
src/web/src/models/organization.ts Introduces interfaces for organization, person, etc.
src/web/src/components/Organizations/Organizations.tsx Implements a basic organizations table and component structure.
src/web/src/api/registration.ts Adds a function to post registration data for an organization.
src/web/src/api/organizations.ts Adds a function to fetch organizations with a hardcoded endpoint.
Files not reviewed (3)
  • src/web/package-lock.json: Language not supported
  • src/web/package.json: Language not supported
  • src/web/src/components/Registration/survey.css: Language not supported
Comments suppressed due to low confidence (1)

src/web/src/api/registration.ts:3

  • Importing the Organization model with the '.ts' extension is non-standard in TypeScript projects and may lead to issues with module resolution. Consider removing the '.ts' extension from the import.
import {Organization} from "../models/organization.ts";

Comment thread src/web/src/api/organizations.ts Outdated
heitzuli and others added 2 commits March 29, 2025 19:28
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@heitzuli heitzuli requested a review from Copilot March 29, 2025 17:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request introduces new components, services, and models to support organization management, which will underpin further enhancements such as styling the registration form. Key changes include:

  • A new OrganizationService class for RESTful operations.
  • Updated Organization model with extended properties.
  • An Organizations component that displays a table of organizations and a new API endpoint for registration.

Reviewed Changes

Copilot reviewed 5 out of 8 changed files in this pull request and generated no comments.

File Description
src/web/src/services/organizationService.ts Introduces a new service to handle Organization REST calls.
src/web/src/models/organization.ts Defines Organization and related interfaces.
src/web/src/components/Organizations/Organizations.ts Implements a table component for organizations.
src/web/src/api/registration.ts Provides a registration API endpoint using axios.
Files not reviewed (3)
  • src/web/package-lock.json: Language not supported
  • src/web/package.json: Language not supported
  • src/web/src/components/Registration/survey.css: Language not supported
Comments suppressed due to low confidence (1)

src/web/src/models/organization.ts:29

  • [nitpick] The property name 'treyMemberCount' is ambiguous. Consider renaming it to a more descriptive identifier to clarify its intent.
29 +  treyMemberCount?: number | null;

@Kitsupitsu Kitsupitsu force-pushed the main branch 2 times, most recently from bb99ed2 to a35d0cd Compare December 17, 2025 10:23
@Kitsupitsu Kitsupitsu force-pushed the main branch 3 times, most recently from b39949f to 487453a Compare January 22, 2026 13:49
@Kitsupitsu Kitsupitsu force-pushed the main branch 2 times, most recently from 954d642 to bb95783 Compare April 8, 2026 11:03
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.

3 participants