Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Full Stack User test#26

Open
deaspo wants to merge 9 commits intoinfinityworks:mainfrom
deaspo:fullstack-test
Open

Full Stack User test#26
deaspo wants to merge 9 commits intoinfinityworks:mainfrom
deaspo:fullstack-test

Conversation

@deaspo
Copy link

@deaspo deaspo commented Jan 25, 2024

As a user of the food ratings app I want to see food hygiene ratings as percentages in a chosen local authority so that I can understand the profile of establishments in that authority
page-navigation
fhrs_authority
authority-list
fhis_authority

- packages installed
    - react-router-dom for routing
    - eslint for linting
    - tailwindcss for styling
    - prettier for code formatting
    - upgraded react-scripts to fix build cra issues
- ignore webstorm idea folder
- tsconfig compile options: target updated to es2015
- additional props, create api method, isEmpty, use search url params
- useEstablishmentsByAuthority.ts - collects all the establishment for a particular local authority
- getFoodRatingsCount.ts - gets the count for all the ratings belonging to a collection of establishments
Establishments
- establishmentsAPI.ts - GET Establishments/basic
- establishmentByIdAPI.ts - GEt Establishments/{id}
- establishmentsByLocalAuthority.ts - GET Establishments?localAuthorityId={localAuthorityId}
Authorities
- authoritiesAPI.ts - GET Authorities/Basic
- authoritiesByIdAPI.ts - GET Authorities/{id}
Scheme
- schemeTypesApi.ts - GET SchemeTypes
- schemeTypeByIdAPI.ts = GEt SchemeTypes/{id}

Removed/renamed files ratingsAPI.ts, ratingsAPI.test.tsx
- DetailAuthorityPage.tsx - detail local authority view
- EstablishmentsTable.tsx - Table with establishment data
- GetFoodRatingsForAuthority.tsx - summary of food rating as a percentage
- HomePage.tsx - added routes
- PaginatedAuthoritiesTable.tsx - Table with local authority data
- PaginatedEstablishmentsTable.tsx - refactored
- establishmentsAPI.test.tsx
- authoritiesAPI.test.tsx
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant