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.

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

add washtub client lib #9

Merged
merged 14 commits into from
Jun 25, 2024
Merged

add washtub client lib #9

merged 14 commits into from
Jun 25, 2024

Conversation

nyelonong
Copy link
Contributor

No description provided.

nyelonong and others added 2 commits January 15, 2024 14:34
Changes:
- Refactor Event
- Fixing some config structure
- Add readme
entities/washtub.go Outdated Show resolved Hide resolved
Copy link
Contributor

@adityayuga adityayuga left a comment

Choose a reason for hiding this comment

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

btw coach, kyknya next enak pakai rebase dari pada merge wkwk

ports:
- "4150:4150"
- "4151:4151"
nsqadmin:
Copy link
Contributor

Choose a reason for hiding this comment

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

gak skalian washtubnya coach? wkwk

Copy link
Contributor Author

Choose a reason for hiding this comment

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

tidak sesimpel itu, image reponhya mau tarok mana? beda pr aja

main.go Outdated Show resolved Hide resolved
client *http.Client
}

func NewWashtuber(ctx context.Context, address string) (washtub.Washtuber, error) {
Copy link
Contributor

Choose a reason for hiding this comment

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

gw jadi ada ide, ini kan client washtub, dibikin pisah repo aja? wkwk

Copy link
Contributor

Choose a reason for hiding this comment

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

kyknya timeout sama pulse intervalnya bisa diconfig in deh coach

Copy link
Contributor

Choose a reason for hiding this comment

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

sama ChannelID better di define waktu new, karena akan selalu fix, jadi gausah di function

Copy link
Contributor Author

Choose a reason for hiding this comment

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

consumer kan bisa banyak, tp init cuma sekali. mau gimana simpan channel id nya?

type Washtuber interface {
Pulse(ctx context.Context, data entities.PulseRequest) chan error

Message(ctx context.Context, data entities.MessageRequest) (*entities.MessageResponse, error)
Copy link
Contributor

@adityayuga adityayuga Mar 31, 2024

Choose a reason for hiding this comment

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

Menurut gw better LogMessage(body string, status string) error

channelID ambil dari washtub property

client *http.Client
}

func NewWashtuber(ctx context.Context, address string) (washtub.Washtuber, error) {
Copy link
Contributor

Choose a reason for hiding this comment

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

sama ChannelID better di define waktu new, karena akan selalu fix, jadi gausah di function

)

type Washtuber interface {
Pulse(ctx context.Context, data entities.PulseRequest) chan error
Copy link
Contributor

Choose a reason for hiding this comment

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

ini better gini gak coach?
Pulse(ctx context.Context) -> address dll ambil dari property washtuber, trus status auto true wkwk

yg ini, masuk ke init washtuber nya:
Address string json:"address"
Topic string json:"topic"
SinkType string json:"sink_type"
Status string json:"status"

@nyelonong nyelonong changed the title [WIP] add washtub client lib add washtub client lib Apr 1, 2024
@nyelonong
Copy link
Contributor Author

btw coach, kyknya next enak pakai rebase dari pada merge wkwk

naini wkwk knp tu yug?

@nyelonong nyelonong merged commit f802033 into main Jun 25, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants