-
Notifications
You must be signed in to change notification settings - Fork 177
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.
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
[Access] Refactor REST Routes Package #6616
base: master
Are you sure you want to change the base?
[Access] Refactor REST Routes Package #6616
Conversation
…drukhiv/6615-rest-refactoring
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6616 +/- ##
===========================================
- Coverage 41.20% 35.79% -5.42%
===========================================
Files 2052 393 -1659
Lines 182191 34996 -147195
===========================================
- Hits 75075 12527 -62548
+ Misses 100824 21362 -79462
+ Partials 6292 1107 -5185
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Closes: #6615
Context
This PR introduces a significant restructuring of the
access/rest
folder, enhancing modularity and clarity.http
andWebSocket
components intohttp
andWebSockets
directories, simplifying maintenance and readability.rest/common
package.RouterBuilder
implementation is separated from all routes and moved torest/router
.http
andws
routes are now inhttp_routes.go
andws_routes.go
, respectively.v1/subscribe_events
route is moved into thewebsocket/legacy
folder to separate futureWebSocket
implementations.