buidl-labs / buidl-labs/YieldScan

Code Audit Todo's

Open
#17 4 comments 0 reactions 0 assignees View on GitHub
-size-m +P3 robustness-problem technical-debt
Dominant language
JavaScript
Stars
11
Forks
6
PR merge metrics
No merged PRs in 30d

Description

- [ ] File structure needs to include folders for assets + stylesheets folder to be shifted in there
- [ ] All components should be wrapped in a folder and at least have component crash tests, ideally also unit tests for functions within the components (this should be followed for all new code going forward).
- [ ] For views like "suggested validators", which have multiple components, create sub-folders and a package.json file to indicate which is the "main" file for easy import
- [ ] Move unnecessary state data outside (i.e. data which is causing unnecessary re-renders)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.