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

I added the endpoints for the todos (CURD) and then i added the tod #7

Closed
wants to merge 58 commits into from
Closed
Show file tree
Hide file tree
Changes from 18 commits
Commits
Show all changes
58 commits
Select commit Hold shift + click to select a range
c70a320
initial spec
Aug 3, 2024
3c5186a
I added the endpoints for the todos (CURD) and then i added the todo-…
IsraelOjiefoh Aug 3, 2024
470f828
updates lock and generate script
Aug 4, 2024
1d4b906
Merge branch 'example' into todo-setup
Aug 4, 2024
f5fcdfa
formatting
Aug 4, 2024
d554557
Merge pull request #1 from elmerbulthuis/todo-setup
IsraelOjiefoh Aug 5, 2024
2e1563e
Enhance TODO API OpenAPI spec: response schemas, request body, path p…
IsraelOjiefoh Aug 5, 2024
0bc2d82
Enhance TODO API OpenAPI spec: response schemas, request body, path p…
IsraelOjiefoh Aug 5, 2024
c9afa11
-Created a new schema todo-item-update which allows for partial updat…
IsraelOjiefoh Aug 6, 2024
87a00e8
- installed the todo-api library -uninstalled node-cqrs and express -…
IsraelOjiefoh Aug 6, 2024
0ebeeac
some changes to the spec
Aug 7, 2024
1dea9d1
update gh actions
Aug 7, 2024
8f3673e
build before test
Aug 7, 2024
1a28458
configure rollup a bit
Aug 7, 2024
1229bf5
update deps
Aug 7, 2024
546af6c
config
Aug 7, 2024
9ee450d
Merge pull request #2 from elmerbulthuis/todo-update
IsraelOjiefoh Aug 8, 2024
14a3475
Added logic for create todo in the operation handlers
IsraelOjiefoh Aug 6, 2024
8b255f0
formatting and update operationhandler
Aug 8, 2024
5a7a8ea
Merge pull request #3 from elmerbulthuis/plishing-up
IsraelOjiefoh Aug 9, 2024
ff89227
Wrote test for the create-todo. It does not work now because it requi…
IsraelOjiefoh Aug 10, 2024
36aa8f9
fixing scripts
Aug 11, 2024
9582b5d
remove explicit types
Aug 11, 2024
655eb40
Merge pull request #5 from elmerbulthuis/types
IsraelOjiefoh Aug 11, 2024
6f919a4
Merge pull request #4 from elmerbulthuis/building
IsraelOjiefoh Aug 11, 2024
95af824
Fixed bug in `todo.test.js` and tried to implement UI for adding todo…
IsraelOjiefoh Aug 12, 2024
bec4801
fix operaiton handler name
Aug 12, 2024
41cfa9d
Merge pull request #6 from elmerbulthuis/renaming-operation-handler
IsraelOjiefoh Aug 12, 2024
0a15445
Added Logic for Update todo
IsraelOjiefoh Aug 16, 2024
0a7e6a2
Added List todo handler
IsraelOjiefoh Aug 16, 2024
7b510d2
Added the delete todo handler
IsraelOjiefoh Aug 17, 2024
86999b3
Created updateTodo and deleteTodo, and also wrote test for UpdateTodo…
IsraelOjiefoh Aug 19, 2024
b612f81
formatting
Aug 22, 2024
1968106
Created Issue in the todo.ts file
IsraelOjiefoh Aug 22, 2024
40f334b
Created Issue In todo.ts(deleteTodoItem)
IsraelOjiefoh Aug 22, 2024
0ba5860
No changes really
IsraelOjiefoh Aug 22, 2024
459742a
Wrote test for List and delete Todo
IsraelOjiefoh Aug 22, 2024
9935d32
implement all handlers (with an error) and formatting
Aug 24, 2024
3753367
fix deletetodo
Aug 24, 2024
2cdb270
adds test scenario
Aug 24, 2024
e3db8de
skip unit tests
Aug 24, 2024
eb855df
6/10 tests passed!
IsraelOjiefoh Aug 30, 2024
14aac72
All tests passed!
IsraelOjiefoh Aug 30, 2024
18025f6
start test server once across all tests
IsraelOjiefoh Aug 30, 2024
c9acb09
formatting
Aug 31, 2024
7ea248a
simplified implementation!
IsraelOjiefoh Sep 1, 2024
b82ea25
Simplified Implementation!
IsraelOjiefoh Sep 1, 2024
fcd9c4f
proper import
Sep 5, 2024
f95d258
Removed Unnecessary Commands
IsraelOjiefoh Sep 1, 2024
1bdddcb
Refactored!
IsraelOjiefoh Sep 7, 2024
f1ac6f5
workspace
IsraelOjiefoh Sep 8, 2024
9878e28
Delete packages/npm/todo/scripts/backEnd.code-workspace
IsraelOjiefoh Sep 8, 2024
2df2c4c
Refactored to use context
IsraelOjiefoh Sep 13, 2024
0bbbf82
small change
Sep 16, 2024
21827d3
Merge branch 'main' into example
Sep 16, 2024
c1a7710
format
Sep 17, 2024
f69a371
update lock
Sep 23, 2024
430fb6e
upgrade
Sep 24, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/style.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ on:
jobs:
formatting:
runs-on: ubuntu-latest
container: node:21.5.0-alpine3.19
container: node:22.6.0-alpine3.19
steps:
- run: apk add make
- uses: actions/checkout@v4
Expand All @@ -13,7 +13,7 @@ jobs:

spelling:
runs-on: ubuntu-latest
container: node:21.5.0-alpine3.19
container: node:22.6.0-alpine3.19
steps:
- run: apk add make
- uses: actions/checkout@v4
Expand Down
19 changes: 5 additions & 14 deletions .github/workflows/test-npm.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,27 +4,18 @@ on:
jobs:
audit:
runs-on: ubuntu-latest
container: node:21.5.0-alpine3.19
container: node:22.6.0-alpine3.19
steps:
- uses: actions/checkout@v4
- run: npm ci
- run: npm --workspaces audit --audit-level high

test-lts:
strategy:
matrix:
node-version: [18, 20, 22]
runs-on: [ubuntu-latest, windows-latest, macos-latest]
runs-on: ${{ matrix.runs-on }}
test:
runs-on: ubuntu-latest
container: node:22.6.0-alpine3.19
steps:
- uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
- run: corepack enable

- uses: actions/checkout@v4
- run: npm run initialize

# node test runner seems to support globs since v21
- if: ${{ matrix.node-version >= 21 }}
run: npm --workspaces test
- run: npm --workspaces test
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
!.gitattributes
!.editorconfig
!.npmrc

node_modules/
target/
coverage/
Expand Down
4 changes: 4 additions & 0 deletions SkiffaExamples.code-workspace
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,10 @@
"name": "npm/reverse",
"path": "packages/npm/reverse",
},
{
"name": "npm/todo",
"path": "packages/npm/todo",
},
],
"settings": {
"window.title": "Skiffa Examples",
Expand Down
1 change: 1 addition & 0 deletions cspell.config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ words:
- remle
- ryanluker
- skiffa
- todos
useGitignore: true
files:
- "**"
Expand Down
Loading
Loading