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.

Sign up for GitHub

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

Jump to bottom

Rh/testing #39

Open
wants to merge 28 commits into
base: dev
Choose a base branch
from
Open

Rh/testing #39

wants to merge 28 commits into from

Conversation

Gambarou
Copy link
Contributor

@Gambarou Gambarou commented Nov 17, 2023 •
edited by lcchrty
Loading

Briefly describe your changes:

Adds limited testing with Jest and React Testing Library.

Fixes UI issues with the admin sign-in page.
Small UI changes to splash page.

Jira issue number and link:

Type of change:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Please answer the following:

1. What is the current behavior?
If the feature did not exist, please say so.
Did not have testing before.
2. What is the new behavior?
There is testing now.
3. What tests did you run?

Checklist before requesting a review

  • I have performed a self-review of my code.
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests for my code.
  • I have resolved all merge conflicts.

@Gambarou
changed invoice page to no longer hit our API route, but to get the s…
2224122 
…tripe API directly
@Gambarou
Added api file to lib folder to hold third party api call logic
2143797
@Gambarou
Added error prop to InvoiceDisplay component
a7d6786
@Gambarou
Small change to Navbar
09791e5
@Gambarou
Fixed pagination issue in the DataTable.
930592a
@Gambarou
Changes to the mappedData prop in overview component
76861c5
@Gambarou
Changes to admin page and utils file
eb58558
@Gambarou
Updated the admin page to be more modular, and separated more of the …
63aca66 
…logic into the utils folder.
@Gambarou
Updated the XAxis datakey to be month instead of name
30c46db
@Gambarou
Cleaned up the utils file and changed the functions to be more concis…
8b737e2 
…e and easier to maintain
@Gambarou
Deleted unused components
37cb5a3
@Gambarou
Deletes unused types in types.ts file
eb61108
@Gambarou
Fixed the most recent sales component so now it actually shows the mo…
6a86cb9 
…st recent sales.
@Gambarou
Changed table headers from buttons to divs
2acbd6b
@Gambarou
Small UI change to InvoiceDisplay
da3b3aa
@Gambarou
Deletes include statement that wasn't being used
4aa56cb
@Gambarou
Deletes unused API route. Changes invoice display to be shadCN compon…
ee154eb 
…ents instead of radix UI. Changes styling in invoice section.
@Gambarou
Adds shadCN badge component in InvoiceDisplay
125f979
@Gambarou
Fixes mobile formatting issue with splash page.
ea37eae
@Gambarou
Small changes
04c0af6
@Gambarou
Deleted old import statement
fa2e211
@Gambarou
Fixes logic for getReveneueByMonth in utils file
43547e3
@Gambarou
Fixes logic with getRevenueByMonth
1ff7b64
@Gambarou
Adds style to globals that overrides the default white background on …
8744814 
…input autofill. Changes UI styling slightly for admin login and adds sorting for payment date in admin dashboard.
@Gambarou
Adds testing with Jest and React Testing Library. Tests added for hom…
867aaf7 
…e page and some for Admin page. Changes in splash page to get rid of button and just use link.
@Gambarou
Fixes some UI issues with the admin sign in page
da646e1
@Gambarou
Few small UI changes
139e190
@vercel Vercel
Copy link

vercel bot commented Nov 17, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
pay-stream ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 17, 2023 9:33am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Gambarou