grassrootseconomics/sarafu-vise
8
0
Fork 0
Code Issues 20 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

ens #32

Merged
lash merged 21 commits from ens into master 2025-03-07 06:00:18 +01:00
Conversation 0 Commits 21 Files Changed 32 +4 -4
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 830d867ac8 - Show all commits

8
handlers/application/menuhandler.go
View File

@ -1476,9 +1476,9 @@ func (h *MenuHandlers) CheckBalance(ctx context.Context, sym string, input []byt
if db.IsNotFound(err) { if db.IsNotFound(err) {
balance := "0.00" balance := "0.00"
if alias != "" { if alias != "" {
content = l.Get("Alias: %s\nBalance: %s\n", alias, balance) content = l.Get("%s balance: %s\n", alias, balance)
} else { } else {
content = l.Get("Balance: %s\n", balance) content = l.Get("balance: %s\n", balance)
} }
res.Content = content res.Content = content
return res, nil return res, nil
@ -1505,9 +1505,9 @@ func (h *MenuHandlers) CheckBalance(ctx context.Context, sym string, input []byt
balStr := fmt.Sprintf("%.2f %s", balFloat, activeSym) balStr := fmt.Sprintf("%.2f %s", balFloat, activeSym)
if alias != "" { if alias != "" {
lash commented 2025-02-28 23:14:14 +01:00
Outdated
Review
Copy Link

DRY?

DRY?
carlos commented 2025-03-03 10:55:30 +01:00
Outdated
Review
Copy Link

Commit: e97d4be296 aims to resolve this. The CheckBalance should possibly be refactored further maybe? Currently it not only loads the balance,but also the user alias if set.

Commit: e97d4be296ee2dd35197b28aabb7487fc3061886 aims to resolve this. The `CheckBalance` should possibly be refactored further maybe? Currently it not only loads the balance,but also the user alias if set.
content = l.Get("Alias: %s\nBalance: %s\n", alias, balStr) content = l.Get("%s balance: %s\n", alias, balStr)
} else { } else {
content = l.Get("Balance: %s\n", balStr) content = l.Get("balance: %s\n", balStr)
} }
res.Content = content res.Content = content