-
Notifications
You must be signed in to change notification settings - Fork 117
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
[IND-288]: Remove unused socks fields and add candle resolutions for websockets #633
Conversation
WalkthroughThe changes primarily focus on simplifying the codebase by consolidating message types, removing unused imports, and adjusting object properties in various functions. The modifications also include the addition of a Changes
TipsChat with CodeRabbit Bot (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Files selected for processing (6)
- indexer/packages/postgres/src/types/websocket-message-types.ts (2 hunks)
- indexer/services/ender/tests/lib/candles-generator.test.ts (2 hunks)
- indexer/services/ender/src/lib/candles-generator.ts (1 hunks)
- indexer/services/socks/tests/helpers/from-kafka-helpers.test.ts (5 hunks)
- indexer/services/socks/src/helpers/from-kafka-helpers.ts (4 hunks)
- indexer/services/socks/src/types.ts (2 hunks)
Files skipped from review due to trivial changes (4)
- indexer/services/ender/src/lib/candles-generator.ts
- indexer/services/socks/tests/helpers/from-kafka-helpers.test.ts
- indexer/services/socks/src/helpers/from-kafka-helpers.ts
- indexer/services/socks/src/types.ts
Additional comments (Suppressed): 4
indexer/services/ender/__tests__/lib/candles-generator.test.ts (2)
480-485: The
CandleColumns.resolution
property is no longer omitted from thecandleMessageContents
object. This change is consistent with the PR summary. However, ensure that this change does not affect other parts of the code that rely on thecandleMessageContents
object.516-521: The
CandleColumns.resolution
property is no longer omitted from theexpectedCandle
object. This change is consistent with the PR summary. However, ensure that this change does not affect other parts of the code that rely on theexpectedCandle
object.indexer/packages/postgres/src/types/websocket-message-types.ts (2)
1-4: The import of
CandleResolution
from./candle-types
is new. Ensure that theCandleResolution
type is correctly defined in thecandle-types
file and is compatible with its usage in theCandleMessageContents
interface.227-234: The
CandleMessageContents
interface now includes aresolution
property of typeCandleResolution
. This is a change from the previous version where theresolution
was not included in theCandleMessageContents
interface. Make sure that all instances whereCandleMessageContents
is used have been updated to include theresolution
property.
Changelist
Test Plan
unit tests
Author/Reviewer Checklist
state-breaking
label.PrepareProposal
orProcessProposal
, manually add the labelproposal-breaking
.feature:[feature-name]
.backport/[branch-name]
.refactor
,chore
,bug
.