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

kafka produce #8664

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
+349 −101
Open

kafka produce #8664

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
fb76fde
kafka
waynebruce0x Nov 26, 2024
99a5699
patch debug
waynebruce0x Nov 26, 2024
ada6ba7
feedback
waynebruce0x Nov 27, 2024
cf1eea9
bridges
waynebruce0x Nov 27, 2024
c4eef3c
cg
waynebruce0x Dec 3, 2024
76afd12
no need for created
waynebruce0x Dec 3, 2024
81b6806
bridges patch
waynebruce0x Dec 3, 2024
734f4c5
cg patches
waynebruce0x Dec 3, 2024
6b1e285
comments
waynebruce0x Dec 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion coins/src/utils/coins3/produce.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ async function produceTopics(

items.map((item) => {
const { symbol, decimals } = item;
if (!symbol || !decimals) return;
if (!symbol || decimals == null) return;
Copy link
Member

Choose a reason for hiding this comment

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

typeof decimals !== 'number' is safer? or it will skip decimals which are string? hmm... maybe we can leave this as is
image

const message: object = convertToMessage(item, topic);
validate(message, topic);
messages.push({ value: JSON.stringify(message) });
Expand Down