Fix bugs related to notification endpoints (#25548)
This commit is contained in:
parent
fdab4e3d84
commit
defd807073
|
@ -777,11 +777,11 @@ func Routes() *web.Route {
|
|||
m.Group("/notifications", func() {
|
||||
m.Combo("").
|
||||
Get(notify.ListNotifications).
|
||||
Put(notify.ReadNotifications, reqToken())
|
||||
Put(reqToken(), notify.ReadNotifications)
|
||||
m.Get("/new", notify.NewAvailable)
|
||||
m.Combo("/threads/{id}").
|
||||
Get(notify.GetThread).
|
||||
Patch(notify.ReadThread, reqToken())
|
||||
Patch(reqToken(), notify.ReadThread)
|
||||
}, tokenRequiresScopes(auth_model.AccessTokenScopeCategoryNotification))
|
||||
|
||||
// Users (requires user scope)
|
||||
|
|
|
@ -183,7 +183,7 @@ func ReadRepoNotifications(ctx *context.APIContext) {
|
|||
if len(qLastRead) > 0 {
|
||||
tmpLastRead, err := time.Parse(time.RFC3339, qLastRead)
|
||||
if err != nil {
|
||||
ctx.InternalServerError(err)
|
||||
ctx.Error(http.StatusBadRequest, "Parse", err)
|
||||
return
|
||||
}
|
||||
if !tmpLastRead.IsZero() {
|
||||
|
|
|
@ -132,7 +132,7 @@ func ReadNotifications(ctx *context.APIContext) {
|
|||
if len(qLastRead) > 0 {
|
||||
tmpLastRead, err := time.Parse(time.RFC3339, qLastRead)
|
||||
if err != nil {
|
||||
ctx.InternalServerError(err)
|
||||
ctx.Error(http.StatusBadRequest, "Parse", err)
|
||||
return
|
||||
}
|
||||
if !tmpLastRead.IsZero() {
|
||||
|
|
Loading…
Reference in a new issue