Skip to content

Commit

Permalink
accept
Browse files Browse the repository at this point in the history
  • Loading branch information
PieWol committed Jun 10, 2024
1 parent 33fcd82 commit ed89093
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions evaluations/tokenguard_2_piewol.md
Original file line number Diff line number Diff line change
@@ -1,26 +1,22 @@
# Evaluation

- **Status:** in progress
- **Status:** accepted
- **Application Document:** https://github.com/w3f/Grants-Program/blob/master/applications/Tokenguard.md
- **Milestone:** 2
- **Previously successfully merged evaluation:** All evaluations by PieWol

**Milestone Number:** 2

**Deliverables**

| Number | Deliverable | Accepted | Link | Evaluation Notes |
| ------ | ----------- | -------- | ---- |----------------- |
| 0a | License |<ul><li>[x] </li></ul>| https://github.com/tokenguardio/dashboard-creator-server/blob/v2.0.3/LICENSE | ok |
| 0b | Documentation |<ul><li>[ ] </li></ul>| https://github.com/tokenguardio/dashboard-creator-server/blob/v2.0.3/README.md | Inline documentation is sometimes sparse but ok overall. The readme needs some work. See general notes. |
| 0b | Documentation |<ul><li>[x] </li></ul>| https://github.com/tokenguardio/dashboard-creator-server/blob/v2.0.3/README.md | Inline documentation is sometimes sparse but ok overall. The readme works now |
| 0c | Testing and Testing Guide |<ul><li>[x] </li></ul>|https://github.com/tokenguardio/dashboard-creator-server/tree/v2.0.5/src/components| also covered by m1 testing guide. Unit tests are passing. |
| 0d | Dockerfile |<ul><li>[x] </li></ul>| https://github.com/tokenguardio/dashboard-creator-server/blob/v2.0.5/Dockerfile | works via docker compose up |
| 1 | API reading mechanism |<ul><li>[x] </li></ul>| https://github.com/tokenguardio/dashboard-creator-server/tree/v2.0.5 | nice, swagger is cool |
| 2 | Report & estimation |<ul><li>[x] </li></ul>| https://github.com/tokenguardio/dashboard-creator-server/tree/v2.0.5 | ok |


## General notes
Still while following the ``getting started`` section the server is not starting up correctly. Futhermore in the readme ``npm start`` is given as a command to startup the server but it isn't working. For me its fine if the whole thing only works via docker-compose but the readme should not contain any guides that are not working. I would suggest to either remove the content from the readme or fix the relevant section.

## Unit test
tests are passing.
Expand Down

0 comments on commit ed89093

Please sign in to comment.