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.

Sign up for GitHub

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

Jump to bottom

BC-5714 - Remove async and context #4696

Closed
wants to merge 30 commits into from
Closed

BC-5714 - Remove async and context #4696

wants to merge 30 commits into from

Conversation

bischofmax
Copy link
Contributor

Description

Links to Tickets or other pull requests

Changes

Datasecurity

Deployment

New Repos, NPM pakages or vendor scripts

Approval for review

  • DEV: If api was changed - generate-client:server was executed in vue frontend and changes were tested and put in a PR with the same branch name.
  • QA: In addition to review, the code has been manually tested (if manual testing is possible)
  • All points were discussed with the ticket creator, support-team or product owner. The code upholds all quality guidelines from the PR-template.

Notice: Please remove the WIP label if the PR is ready to review, otherwise nobody will review it.

bischofmax and others added 29 commits January 2, 2024 15:37
@bischofmax
Add fileExists property to fileRecord
3c704ad
@bischofmax
Add fileExists to response
59972ce
@bischofmax
Add isUploading
4548b8f
@bischofmax
Add tests
2798ef1
@bischofmax
Add promise resolve to test mocks
5eec031
@bischofmax
Use ApiPropertyOptional for isUploading prop
304624e
@bischofmax
Merge branch 'main' into BC-5714-store-upload-state
356b013
@bischofmax
Add apiheader for If-None-Match
7817516
@bischofmax
Merge branch 'BC-5714-store-upload-state' of github.com:hpi-schul-clo…
b49ed3f 
…ud/schulcloud-server into BC-5714-store-upload-state
@bischofmax
Add console logs
aec41e4
@bischofmax
Merge branch 'main' into BC-5714-store-upload-state
e9d0c86
@bischofmax
Revert "Add console logs"
37ab24d 
This reverts commit aec41e4.
@bischofmax
Turn on mikroorm debug
4e5697a
@SevenWaysDP
BC-5714 - add req context
1489825
@bischofmax
Merge branch 'main' into BC-5714-store-upload-state
6276908
@bischofmax
Rename markAsLoaded to markAsUploaded
49aab22
@bischofmax
Remove mikororm debug mode
c01c009
@SevenWaysDP
Merge branch 'BC-5714-store-upload-state' of https://github.com/hpi-s…
3f75e03 
…chul-cloud/schulcloud-server into BC-5714-store-upload-state
@SevenWaysDP
BC-5714 - add ingress config
b5cc099
@SevenWaysDP
Revert "BC-5714 - add req context"
2bcff0c 
This reverts commit 1489825.
@SevenWaysDP
BC-5714 - remove await
fdc2e71
@SevenWaysDP
add error logging
970f94a
@SevenWaysDP
try request scope
ba492b4
@bischofmax
Add scope to repo
e9fd73f
@SevenWaysDP
add scope to MikroOrm Module
c6027ea
@bischofmax
Create new async request context in uc
336ba02
@bischofmax
Remove await from busboy on file
e1e3a1c
@bischofmax
Only starte file storage api server on debug command
c4eb4fe
@bischofmax
Remove RequestContext.create
4df2220
@bischofmax bischofmax self-assigned this Jan 15, 2024
@bischofmax bischofmax closed this Jan 15, 2024
@bischofmax bischofmax deleted the BC-5714-remove-async-and-context branch January 15, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@bischofmax bischofmax

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bischofmax @SevenWaysDP