-
Notifications
You must be signed in to change notification settings - Fork 4
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
User payment #15
base: master
Are you sure you want to change the base?
User payment #15
Conversation
ready |
conflicted |
8215246
to
40cd926
Compare
ready |
res.status(404).json({ message: 'User not found' }); | ||
return; |
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.
res.status(404).json({ message: 'User not found' }); | |
return; | |
return res.status(404).json({ message: 'User not found' }); |
res.status(404).json({ message: 'Course not found' }); | ||
return; |
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.
res.status(404).json({ message: 'Course not found' }); | |
return; | |
return res.status(404).json({ message: 'Course not found' }); |
res.status(404).json({ message: "Payment not found or not authorized" }); | ||
return; |
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.
sửa tương tự e nha
I18n các message nữa nhé
Checklist tự review pull trước khi ready để trainer review
Related Tickets
- ticket redmine
WHAT (optional)
completed/total
in admin page.HOW
WHY (optional)
normal
items. But in new version, we havestate
andconfirm_state
depends on bothnormal
+not_normal
items.Evidence (Screenshot or Video)
Notes (Kiến thức tìm hiểu thêm)