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

Create data layer integration for /admin-menu endpoint #45464

Closed
3 of 5 tasks
getdave opened this issue Sep 8, 2020 · 0 comments · Fixed by #45495
Closed
3 of 5 tasks

Create data layer integration for /admin-menu endpoint #45464

getdave opened this issue Sep 8, 2020 · 0 comments · Fixed by #45495
Assignees
Labels
[Feature] Calypso & wp-admin Navigation All navigation in Calypso and wp-admin, and the unified transitions between the two.

Comments

@getdave
Copy link
Contributor

getdave commented Sep 8, 2020

Sub Issue of #45435.

This documents the need to (experiment with) creating a data-layer and state implementation for the admin-menu endpoint proposed in D49005-code.

Tasks

  • Create new admin-menu directory in client/state/data-layer/wpcom.
  • Define Redux action namespace (eg: ADMIN_MENU_).
  • Register handler module for all menu items (registerHandlers).
  • Wire up basic WPCom HTTP request using dispatchRequest and logging from the various response actions creators.
  • Write data transformer to transform the API response into consumble data.

Following this, the next step will be to create a state tree for admin menus with actions, reducers and selectors to make the data available to the UI.

@getdave getdave self-assigned this Sep 9, 2020
@getdave getdave added the [Feature] Calypso & wp-admin Navigation All navigation in Calypso and wp-admin, and the unified transitions between the two. label Sep 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Calypso & wp-admin Navigation All navigation in Calypso and wp-admin, and the unified transitions between the two.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant