Fynn Flügge
Fynn Flügge
Currently the electron app is not build and published to S3 during workflows. Add electron build step in `Build` job and upload the files to S3 in `Deploy` job.
If a shared document is opened in Whatsapp, Slack etc. it should show a link preview with image, title and description. Lambda Function as Proxy to S3 Static Website needed.
[sidenav.component.ts](https://github.com/fynnfluegge/rocketnotes/blob/main/webapp/src/app/component/navigation/sidenav.component.ts) needs some refactor and documentation comments since the whole document tree logic resides in this class and the complexity grew over time.
Setting up local dev environment can be accelerated by providing a devcontainer.
CI [pipeline](https://github.com/fynnfluegge/rocketnotes/blob/main/.github/workflows/main.yml) should test the web app triggered by any pull request. A test stage should be added to the pipeline.
When entering three characters in the search bar and hitting space afterwards, then the autocompleted result list is not displayed correctly. It seems that the [search function](https://github.com/fynnfluegge/rocketnotes/blob/main/lambda-handler/search-document-handler/main.go) returns wrong results...
Add a Unit test for [get-document-tree-handler](https://github.com/fynnfluegge/rocketnotes/tree/main/lambda-handler/get-document-tree-handler). See this [guide](https://adityarama1210.medium.com/golang-mocking-a-function-for-unit-testing-497b43ad3409) how to mock DynamoDB methods.
Add a Unit test for [get-document-handler](https://github.com/fynnfluegge/rocketnotes/tree/main/lambda-handler/get-document-handler). See this [guide](https://adityarama1210.medium.com/golang-mocking-a-function-for-unit-testing-497b43ad3409) how to mock DynamoDB methods.
- Remove `Chunk header` starting by `@@` and replace by the related filename. - Remove chunks with only removed lines.