-
Notifications
You must be signed in to change notification settings - Fork 42
feat: export Plans page as a component and consume it in apps/admin #1376
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
base: feat/export-admins-page
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
Pull Request Test Coverage Report for Build 21901150617Details
💛 - Coveralls |
…mproved performance
Summary
apps/admintolib/admin/views/plans/usinggit mvwhere applicable to preserve history.PlansViewfrom@raystack/frontier/adminas a router-agnostic view (same behaviour: list plans, client-side table with search/sort/filter, "Plans" header, plan detail in a sheet whenselectedPlanIdis set).PlansPagewrapper inapps/adminthat renders<PlansView selectedPlanId={planId} onCloseDetail={() => navigate("/plans")} />.plansandplans/:planIdroutes to usePlansPage; route paths and behaviour unchanged.Test plan
/plans/:planIdand opens the detail sheet./plans) returns to the list.pnpm buildinweb/lib.pnpm buildinweb/apps/admin.