Automattic / Automattic/wp-calypso

Plans Next: Remove `allFeaturesList` prop & `getPlanFeaturesObject` function from plans-grid-next

Open
#88,554 0 comments 0 reactions 0 assignees View on GitHub
Plans: NPM essential
Dominant language
TypeScript
Stars
12.6k
Forks
2k
Avg merge
1d 17h
Merged PRs (30d)
552

Description

### What

The `allFeaturesList` prop passed to the grid components and the `getPlanFeaturesObject` function were originally defined to let us create a package while the features-list definitions remained locally in Calypso.

With https://github.com/Automattic/wp-calypso/issues/79004 done, features-list and accessor methods now live in `@automattic/calypso-products`. We can proceed to remove the above concepts and import from the `@automattic/calypso-products` package.

### Why

Clean up temporary and redundant properties and local methods in `plan-grid-next` package.

### How

_No response_

Contributor guide

Open the contributing guide

Research direction

Start in the plans-grid-next package by locating references to the allFeaturesList prop and getPlanFeaturesObject function across the grid components. Trace the existing features-list access through @automattic/calypso-products, remove the redundant local concepts, and verify that the package still builds and its existing checks pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.