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

Nova 4 upgrade #45

Merged
merged 2 commits into from
Aug 12, 2022
Merged

Nova 4 upgrade #45

merged 2 commits into from
Aug 12, 2022

Conversation

faxblaster
Copy link
Contributor

@faxblaster faxblaster commented Aug 12, 2022

Upgrades the package to work with Nova version 4.
Note: Requires new major release

Additionally:

  • Adds visual support for disputed charges
  • Adds translation support

Screenshots

Charges
charges-index

Charge Detail
charges-detail

Customers
customers-index

Customer Detail
customers-detail

update base files

update filters & imports

fixes column selection

import tailwind 3

remove unused resource

update routes

Add Detail page and extract filter

fix column select dropdown styles

update `ChargesTable` styles

Fix titles

update icon

Add customers page to menu

update `ChargeDetail` page styles

Add back title

update component casing

Update `CustomersTable` styles

Remove unused navigation

Update `CustomerDetail` page

Formatting/cleanup

run prettier

extract tables to components

auto-import components

update props to objects

Bugfix for no currency or shipping address

extract layout to component

remove legacy routes

fix meta title

move loading components to `BaseTable`

move pagination to base table

Fix dropdown positioning

Make refunded charges gray

update PHP version, workflows

production

update matrix

update key reference

fix sniffs

add laravel back to matrix

simplify dependencies

add localization

add lang template

fix shipping address

downgrade testbench for laravel 8

remove unused file

fix address

move tests to stripe-mock

run prettier & prod
Copy link

@krievley krievley left a comment

Choose a reason for hiding this comment

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

Looks good 😄

@faxblaster faxblaster merged commit a309089 into main Aug 12, 2022
@faxblaster faxblaster deleted the alk/nova-4-upgrade branch August 12, 2022 18:14
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