Skip to content

feat(TaskProcessing): Allow defining custom task types#270

Merged
bigcat88 merged 4 commits intomainfrom
feat/task-processing/custom-task-types
Jul 13, 2024
Merged

feat(TaskProcessing): Allow defining custom task types#270
bigcat88 merged 4 commits intomainfrom
feat/task-processing/custom-task-types

Conversation

@provokateurin
Copy link
Copy Markdown
Contributor

Counter part for nextcloud/app_api#324

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 11, 2024

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 94.69%. Comparing base (000819d) to head (09900fa).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #270      +/-   ##
==========================================
- Coverage   94.72%   94.69%   -0.04%     
==========================================
  Files          47       47              
  Lines        5403     5405       +2     
==========================================
  Hits         5118     5118              
- Misses        285      287       +2     
Files Coverage Δ
nc_py_api/ex_app/providers/task_processing.py 40.69% <60.00%> (-0.97%) ⬇️

bigcat88 and others added 2 commits July 13, 2024 21:59
Signed-off-by: Alexander Piskun <[email protected]>
Signed-off-by: Alexander Piskun <[email protected]>
Copy link
Copy Markdown
Contributor

@bigcat88 bigcat88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As currently we did not define classes and enums(and probably will not until API in Server will be in final mature state) - I tested with the simple "custom TaskTypes" and all are working.

Thanks @provokateurin for the PR

@bigcat88 bigcat88 merged commit 22cbea9 into main Jul 13, 2024
@bigcat88 bigcat88 deleted the feat/task-processing/custom-task-types branch July 13, 2024 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants