Skip to content

Bugfix/lesson 6 - #1

Open
emxqm-11 wants to merge 4 commits into
Duomly:Golang-course-Lesson-6from
emxqm-11:bugfix/lesson-6
Open

Bugfix/lesson 6#1
emxqm-11 wants to merge 4 commits into
Duomly:Golang-course-Lesson-6from
emxqm-11:bugfix/lesson-6

Conversation

@emxqm-11

@emxqm-11 emxqm-11 commented Jan 9, 2021

Copy link
Copy Markdown

In main.go:

  • fix migrations.Migrate() and removed migrations.MigrateTransaction() which is nonexistent
  • CreateTransaction was missing Amount so the logs for transactions would always show 0 for amount recorded in postgresql
  • i added a lot of debugging errors i got to README so it can be helpful to others.
  • function Transaction()'s updateAccount logic for adding amount to the toAccount was wrong. It used the fromAccount by mistake. Line 56.

@emxqm-11

emxqm-11 commented Jan 9, 2021

Copy link
Copy Markdown
Author

@radoslawfabisiak Let me know if you want any changes, i know the README was a big change but the rest were bug fixes. Also great work on this tutorial, i loved it ! Hence I wanted to contribute.

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.

1 participant