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

Created a skeleton AdminX demo app #19005

Merged
merged 13 commits into from
Nov 20, 2023
Merged

Created a skeleton AdminX demo app #19005

merged 13 commits into from
Nov 20, 2023

Conversation

binary-koan
Copy link
Contributor

@binary-koan binary-koan commented Nov 15, 2023

refs https://github.com/TryGhost/Product/issues/4152


🤖[deprecated] Generated by Copilot at a28462f

This pull request adds a new admin-x app called admin-x-demo, which demonstrates how to use the shared packages admin-x-framework and admin-x-design-system to create a simple app that renders a button and a modal. It also improves the development workflow, the vite integration, the dependency management, and the type checking for the admin-x apps and packages. It modifies some files in the admin-x-framework and admin-x-design-system packages to make the modals prop optional, to introduce a new type for the props from the Ember app, to fix the z-index of the modal backdrop, and to use consistent file extensions and module syntax.

@binary-koan binary-koan self-assigned this Nov 15, 2023
Copy link

codecov bot commented Nov 16, 2023

Codecov Report

Attention: 80 lines in your changes are missing coverage. Please review.

Comparison is base (b4fa565) 72.77% compared to head (bb661ea) 72.76%.
Report is 8 commits behind head on main.

Files Patch % Lines
.../admin/app/components/admin-x/admin-x-component.js 9.19% 75 Missing and 4 partials ⚠️
ghost/admin/app/components/admin-x/settings.js 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #19005      +/-   ##
==========================================
- Coverage   72.77%   72.76%   -0.01%     
==========================================
  Files        1251     1253       +2     
  Lines       72611    72627      +16     
  Branches     9602     9604       +2     
==========================================
+ Hits        52840    52849       +9     
- Misses      18989    18997       +8     
+ Partials      782      781       -1     
Flag Coverage Δ
admin-tests 40.96% <14.89%> (+<0.01%) ⬆️
e2e-tests 81.54% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions github-actions bot added the affects:admin Anything relating to Ghost Admin label Nov 20, 2023
@binary-koan binary-koan merged commit a93c665 into main Nov 20, 2023
22 checks passed
@binary-koan binary-koan deleted the admin-x-demo-app branch November 20, 2023 13:30
cmraible pushed a commit to cmraible/Ghost that referenced this pull request Nov 21, 2023
refs TryGhost/Product#4152

---

<!-- Leave the line below if you'd like GitHub Copilot to generate a
summary from your commit -->
<!--
copilot:summary
-->
### <samp>🤖[[deprecated]](https://githubnext.com/copilot-for-prs-sunset)
Generated by Copilot at a28462f</samp>

This pull request adds a new admin-x app called `admin-x-demo`, which
demonstrates how to use the shared packages `admin-x-framework` and
`admin-x-design-system` to create a simple app that renders a button and
a modal. It also improves the development workflow, the vite
integration, the dependency management, and the type checking for the
admin-x apps and packages. It modifies some files in the
`admin-x-framework` and `admin-x-design-system` packages to make the
modals prop optional, to introduce a new type for the props from the
Ember app, to fix the z-index of the modal backdrop, and to use
consistent file extensions and module syntax.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects:admin Anything relating to Ghost Admin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant