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

feat(tasks): several small updates #270

Merged
merged 2 commits into from
Apr 3, 2024
Merged

feat(tasks): several small updates #270

merged 2 commits into from
Apr 3, 2024

Conversation

guergabo
Copy link
Contributor

@guergabo guergabo commented Apr 3, 2024 •
edited
Loading

Fixes #253
Fixes #254

Changes:

  1. Separate routing information into it's own entity:
  2. Added baseURL configuration to give worker's absolute URL.
  3. Put queue in HTTP body for optional usage and consistency in configuration / queueing semantics.
  4. Fixed small migrations table bug.
  5. Queueing subsytem can only have 1 worker.
Screenshot 2024-04-03 at 1 56 50 PM

Notes:

A bit of the code restructure is not directly necessary for these change, but puts us a step closer to when we support various types of routing (not just regex pattern matching).

@guergabo guergabo added the enhancement New feature or request label Apr 3, 2024
@guergabo guergabo self-assigned this Apr 3, 2024
@guergabo guergabo merged commit 0bc15b1 into main Apr 3, 2024
3 checks passed
@guergabo guergabo deleted the tasks-devex-plus branch April 3, 2024 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@guergabo guergabo

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Improve DevEx of Task Framework Add baseUrl to config
1 participant
@guergabo