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

feat!: refactor Admin UX/UI #141

Closed
wants to merge 25 commits into from
Closed

feat!: refactor Admin UX/UI #141

wants to merge 25 commits into from

Conversation

justlevine
Copy link
Member

@justlevine justlevine commented Sep 29, 2024

What

This PR contains a complete refactor of packages/admin, along with the storage/handling of plugin settings.

More specifically:

Todo:

  • Add path aliases to tsconfig
  • Rename file structure to use kebab-case
  • Clean up components/layouts/screen
  • Redesign header
  • Add custom settings store + REST endpoint
  • Refactor settings schema + screens for composability (DRY/SOLID)
  • Refactor settings registration
  • Settings migration on update

Why

Unblocks the UX issues + scalability for #129

How

Testing Instructions

Additional Info

Checklist:

  • My code is tested to the best of my abilities.
  • My code follows the WordPress Coding Standards.
  • My code has proper inline documentation.
  • I have added unit tests to verify the code works as intended.
  • I included the relevant changes in CHANGELOG.md

@coveralls
Copy link

coveralls commented Sep 29, 2024

Coverage Status

coverage: 71.653% (-10.3%) from 81.924%
when pulling 46ae7a1 on feat/refactor-admin
into 72936f4 on develop.

@justlevine justlevine marked this pull request as ready for review October 20, 2024 19:50
@justlevine justlevine closed this Oct 20, 2024
@justlevine justlevine deleted the feat/refactor-admin branch October 20, 2024 22:10
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