squid/blog
1
2
Fork 0
Code Issues 11 Pull Requests 1 Actions Projects 1 Releases 2 Wiki Activity

BLOG-56 refactor: rename as xxx_handler in routes #57

Open
squid wants to merge 3 commits from BLOG-56_align_clean_architecture into main
pull from: BLOG-56_align_clean_architecture
merge into: squid:main
Conversation 0 Commits 3 Files Changed 4 +22 -22
squid commented 2025-07-01 11:36:20 +08:00
Owner
Copy Link

Description

  • As the description in the issue

    • Use case should be stateless

      The value unwrapped from web::Data must be Arc type

    • Initializing shouldn't be done in Container
    • Rename the functions as xxx_handler in routes

Package Changes

No response

Screenshots

No response

Reference

Resolves #56

Checklist

  • A milestone is set
  • The related issuse has been linked to this branch
### Description - As the description in the issue > - ~~Use case should be stateless~~ > > The value unwrapped from `web::Data` must be `Arc` type > - Initializing shouldn't be done in Container > - Rename the functions as xxx_handler in routes ### Package Changes _No response_ ### Screenshots _No response_ ### Reference Resolves #56 ### Checklist - [x] A milestone is set - [x] The related issuse has been linked to this branch
squid added this to the 0.2.0 milestone 2025-07-01 11:36:20 +08:00
squid added 1 commit 2025-07-01 11:36:20 +08:00
BLOG-56 refactor: rename as xxx_handler in routes
Some checks failed
Frontend CI / build (push) Successful in 1m27s
Details
PR Title Check / pr-title-check (pull_request) Failing after 16s
Details
9dcefe90c5
squid added 1 commit 2025-07-01 11:39:55 +08:00
BLOG-56 refactor: make use cases stateless
Some checks failed
Frontend CI / build (push) Successful in 1m27s
Details
PR Title Check / pr-title-check (pull_request) Failing after 16s
Details
5ea5879ad7
squid force-pushed BLOG-56_align_clean_architecture from 5ea5879ad7 to 6a2c35a74a 2025-07-01 12:24:29 +08:00 Compare
squid force-pushed BLOG-56_align_clean_architecture from 6a2c35a74a to 1dc8ca742b 2025-07-01 12:32:33 +08:00 Compare
squid added 1 commit 2025-07-01 12:46:18 +08:00
BLOG-56 refactor: remove redoundent clone
Some checks failed
Frontend CI / build (push) Successful in 1m27s
Details
PR Title Check / pr-title-check (pull_request) Failing after 16s
Details
b1b8fab108
squid changed title from WIP: BLOG-56 refactor: rename as `xxx_handler` in routes to BLOG-56 refactor: rename as `xxx_handler` in routes 2025-07-01 12:47:23 +08:00
zoe was assigned by squid 2025-07-01 12:47:28 +08:00
Some checks failed
Frontend CI / build (push) Successful in 1m27s
Required
Details
PR Title Check / pr-title-check (pull_request) Failing after 16s
Required
Details
This pull request doesn't have enough required approvals yet. 0 of 1 official approvals granted.
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin BLOG-56_align_clean_architecture:BLOG-56_align_clean_architecture
git checkout BLOG-56_align_clean_architecture
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
design
dev
automation
dev
backend
dev
frontend
dev
improvement
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone 0.2.0
Projects
Clear projects
No project
Assignees
Clear assignees
zoe squid
No Assignees zoe
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: squid/blog#57
No description provided.
Delete Branch "BLOG-56_align_clean_architecture"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?