Skip to content

Исправил недочеты ...1#1

Open
berduk-dev wants to merge 7 commits into
mainfrom
lesson-1
Open

Исправил недочеты ...1#1
berduk-dev wants to merge 7 commits into
mainfrom
lesson-1

Conversation

@berduk-dev
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown
Collaborator

@azdaev azdaev left a comment

Choose a reason for hiding this comment

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

Все четко. Я еще подумал. Когда будем пост открывать на сайте, мы же должны будем все комменты увидеть об этом посте. То есть нам нужен метод для получения всех комментов одного блога

Copy link
Copy Markdown
Collaborator

@azdaev azdaev left a comment

Choose a reason for hiding this comment

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

Код очень хороший. Видно что ты уже потихоньку разбираешься в написании бэкенда. Но пока есть ошибки из-за невнимательности (это не страшно) и есть подозрение что еще не разобрался как работает UPDATE и что такое RETURNING. Это надо поискать и почитать

Comment thread main.go Outdated
Comment thread internal/model/model.go Outdated
Comment thread internal/model/model.go Outdated
Comment thread internal/model/model.go Outdated
Comment thread internal/repo/repo.go Outdated
Comment thread internal/repo/repo.go Outdated
Comment thread internal/repo/repo.go Outdated
Comment thread internal/repo/repo.go Outdated
Copy link
Copy Markdown
Collaborator

@azdaev azdaev left a comment

Choose a reason for hiding this comment

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

Замечательно)

Comment thread internal/handler/handler.go Outdated
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