-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat(?sum): add templates feature #79
base: main
Are you sure you want to change the base?
Conversation
pkg/discord/command/sum/sum.go
Outdated
var data *model.Sum | ||
var err error | ||
|
||
switch len(message.ContentArgs) { |
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.
cái template mình có thể để cụ thể nó là: short/detail hay gì gì đó được khum?
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.
ah vậy thì enum luôn
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.
yeah. với làm nó không phụ thuộc vào vị trí của cái args mình điền vào thì hay hơn.
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.
aight aight done ser 👍
pkg/discord/command/sum/sum.go
Outdated
var data *model.Sum | ||
var err error | ||
|
||
switch len(message.ContentArgs) { |
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.
yeah. với làm nó không phụ thuộc vào vị trí của cái args mình điền vào thì hay hơn.
1773108
to
e0f260e
Compare
914f028
to
f74b30b
Compare
What's this PR do?
?sum