Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Create state to store the API responses for dashboard - Closes #6241, #6242, #6243 #6326

Merged
merged 8 commits into from
Apr 19, 2021
Merged

Create state to store the API responses for dashboard - Closes #6241, #6242, #6243 #6326

merged 8 commits into from
Apr 19, 2021

Conversation

nazarhussain
Copy link
Contributor

What was the problem?

This PR resolves #6241, #6242, #6243

How was it solved?

  • Created react states to store data
  • Created API client to and connect to backend
  • Linked states to components

How was it tested?

  • Manually tested bu running lisk-core with --api-ws

@nazarhussain nazarhussain self-assigned this Apr 15, 2021
This was linked to issues Apr 15, 2021
Create functions to update states using API client #6241
Closed
Create state to store the API responses for dashboard #6242
Closed
Connect states in the main page #6243
Closed
@shuse2 shuse2 requested a review from ManuGowda April 16, 2021 09:19
ManuGowda
ManuGowda reviewed Apr 16, 2021
View reviewed changes
Copy link
Contributor

@ManuGowda ManuGowda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

framework-plugins/lisk-framework-dashboard-plugin/src/ui/types.ts Show resolved Hide resolved
@ManuGowda ManuGowda merged commit d038109 into feature/6216-add_dashboard-plugin Apr 19, 2021
@ManuGowda ManuGowda deleted the 6242-react-state-store branch April 19, 2021 11:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ManuGowda ManuGowda ManuGowda approved these changes

@shuse2 shuse2 shuse2 approved these changes

Assignees

@nazarhussain nazarhussain

Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@nazarhussain @shuse2 @ManuGowda