Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added modifications to report files #511

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

carmen-chau
Copy link
Contributor

@carmen-chau carmen-chau commented Oct 1, 2023

Overview

This PR (replacement of #510, which contained issue regarding showing commits from previous branches) contains a working implementation of adding broken and lost items tables to the Reports page (on the HSS admin side). Currently, the tables are using Mock data.

Note: The page contains the text "IEEEEE" in order to pass the original tests contained in file Reports.test.js. This "placeholder text" will be removed once tests are created for the implemented tables

Unit Tests Created

  • N/A, kept all original tests

Steps to QA

  • Would appreciate any feedback on improving code syntax, suggestions on improving accumulation of entries for display in the "Total" rows of the table, or other general feedback

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