Feedback - Txai-HvA/BlokTech GitHub Wiki
Peer review voor A1 van Project Tech van Akshay Kumar.
Checklist
| Requirement | Comment |
|---|---|
| Source code is available on GitHub | Check |
| Project is documented and has a readme.md | Check, wiki aanvullen |
| Cites the sources used; code comments or docs | Nog aanvullen |
| Research is documented in the wiki | Nog doen |
| A live version is deployed and linked in description | X |
Project install
| Requirement | Comment |
|---|---|
| Yes, the project works without technical problems! | X |
| Kinda, the project runs but gives errors. | |
| No, because reasons; |
1. Concept
Job Story & Requirements List
You've written a good job story, there is a clear concept and there are additional requirements
Interface (wireframes)
The wireframe and screens are incomplete (no states, flow is unclear)
Comments
- Wiki aanvullen
- Code rescourcen en commenten
2. Research
| Requirement | Comment |
|---|---|
| Is there any documentation on the git strategy? | Moet nog |
| Is there any documentation on the local dev environment? | Moet nog |
| Is there any documentation on deploying and production? | Moet nog |
Comments
Nog je menu aanvullen in de wiki met alle content.
3. Application
The feature partially works but is not complete; incomplete flow
Comments
Je app staat er maar is nog niet functioneel, dit moet nog uitgevoerd geworden. Ik neem het filteren.
4. Quality
| Requirement | Comment |
|---|---|
| Does the readme.md cover what the project does? (concept) | Nog uitbreiden |
| Are there any linters installed and used in the project? | Check |
| Is version control user properly? Commits early and often. | Check |
Code quality
The code is readable and consistent, uses modern JavaScript syntax and uses a linter to adhere to code standards
Documentation
The code, project, and process are partially documented
Comments
Nog heel veel aanvullen maar nvm daar ben je al op de hoogte van.