Project

General

Profile

Actions

Refactor #28362

closed

Fix API Middleware failure payload

Added by Redmine Asana Bot over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
JavaScript stack
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:
Actions #1

Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7211 added
Actions #2

Updated by Tomer Brisker over 4 years ago

  • Category set to JavaScript stack
  • Assignee set to Ron Lavi

What failure? when?
Issues with no info aren't very useful when they just appear here from a bot so there is no one to ask for further information.
Noone outside of the asana users know if someone is already working on an issue or try to fix them before they are copied from asana, and the reviewer has no background to what the PR is trying to fix.

Actions #3

Updated by Ron Lavi over 4 years ago

From: `payload: { error, payload }`
To: `payload: { ...payload, error }`

the `payload` data will be accessed in the `reducer` as `payload` instead of `payload.payload`

Actions #4

Updated by Ron Lavi over 4 years ago

Sorry about that, we started to use the bot and I forgot to add some description before it will generate it automatically.
Update the info above,
basically the change is in the payload object which is being accessed in the reducer

Actions #5

Updated by Tomer Brisker over 4 years ago

The comment you added describes the change you did in the pr, not why it is needed, what is the bug it is trying to fix or how it can be reproduced and tested.

Actions #6

Updated by Ron Lavi over 4 years ago

  • Tracker changed from Bug to Refactor

thanks, actually it is more of a refactoring, not a bug
so it will look cleaner to access the payload of an API call failure:

From: `payload: { error, payload }`
To: `payload: { ...payload, error }`

the `payload` data will be accessed in the `reducer` as `payload` instead of `payload.payload`

Actions #7

Updated by The Foreman Bot over 4 years ago

  • Fixed in Releases 2.0.0 added
Actions #8

Updated by Ron Lavi over 4 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF