Skip to content

Commit 638cbfb

Browse files
authored
Sample App: Code Clean up (knative#5948)
* Fix the content in the bad word filter knative function code * Remove the unused import * Adding the action for the button in the front end, so that the new review comment can be delivered to the review service. * enable cors * Install cors * fix: configure node.js so that it won't be restricted by CORS policy * fix: add hardcoded the address for the review service * feat: the yaml files to run the front end in the k8s cluster * fix: fix the wrong command in the docker file * fix: change the naming convention * fix: update knative slack sink to use the value from secret * fix: change the name for the front end application package * fix: update the event type for the outputs of the ML workflows * fix: create the sequence and trigger for the ML workflow * fix: change the input key to reviewText * fix: change the event type for the input cloudevent * fix: set up the trigger that will filter out the bad comment and send them to the event display * feat: add the loading state for the front end * feat: expose the postgresql service as headless service, so we can use the statefulset's name as the host url * fix: format fix, and also make sure the sentiment analysis result is all lower case * feat: update to use websocket for the comment area * feat: add Kuack to the bookstore as advised by UX WG * feat: add kuack to the bookstore * feat: add the new timing format and also set up the status badge * feat: make sure all the ML output are lower case * feat: styling change * feat: add the database insertion logic * fix: update to use kafka broker * feat: add the quickstart setup script * fix: use deployment instead of knative service * fix: Send the reply as cloud event back to broker * fix: Go back to use MT broker * fix: Restructure the code * fix: Restructure the code * Save progress: TODO: will confirm whether use the pre-built images or build the images locally * fix: change the naming of the files * fix: cut the pulling rate of the websocket * fix: front end clean up and change the error message * fix: change the port to 8080 * fix: fix the setup script * fix: update the front end port to use 80 as the nodeserver, as trigger default to use 80, instead of 8080. * fix: trigger default to use 80, and cannot use customized port 8080. * fix: renaming and use the pre-built image * fix: update the naming and add comments to the yaml * feat: move the code to solution folder * feat: add the start folder * fix: the deployment file is not in the /start folder * fix: shorten the bad word filter name * fix: shorten the bad word filter name * fix: delete the deployment files for ML workflows * fix: add the port 8080 * fix: use Service instead of deployment when setting up temp trigger * fix: add the missing deployment yaml for /start * fix: add the shortcut script in the /start folder * fix: change the favicon to kuack to improve the reader's experience
1 parent 30e769c commit 638cbfb

File tree

128 files changed

+11735
-477
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

128 files changed

+11735
-477
lines changed

Diff for: .gitignore

+1-2
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ hack/__pycache__
2020
!/blog/overrides/partials/source-file.html
2121
!/blog/overrides/partials/content.html
2222
/blog/docs/stylesheets/
23-
/node_modules/
23+
node_modules
2424
venv/
2525

2626
# TODO clean up images copied between blog/ and docs/
@@ -36,4 +36,3 @@ maven-wrapper.jar
3636

3737
# Ignore Python virtual environments
3838
.venv
39-
/code-samples/eventing/bookstore-sample-app/db/bookstore-eda/node_modules/*

Diff for: code-samples/eventing/bookstore-sample-app/ML-inappropriate-language-filter/func.yaml

-4
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/ML-sentiment-analysis/func.py

-50
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/ML-sentiment-analysis/func.yaml

-11
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/db/bookstore-eda/broker.yaml

-4
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/db/bookstore-eda/deployment.yaml

-21
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/db/bookstore-eda/event-display.yaml

-9
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/db/bookstore-eda/index.js

-53
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/db/bookstore-eda/service.yaml

-12
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/db/bookstore-eda/trigger.yaml

-34
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/db/sample.sql

-13
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/frontend/client/components/CommentDisplay.js

-38
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/frontend/client/components/CommentForm.js

-44
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/frontend/client/components/CommentList.js

-12
This file was deleted.

Diff for: code-samples/eventing/bookstore-sample-app/frontend/client/pages/Main.js

-37
This file was deleted.

0 commit comments

Comments
 (0)