Refactor #28362
closed
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/7211 added
- 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.
From: `payload: { error, payload }`
To: `payload: { ...payload, error }`
the `payload` data will be accessed in the `reducer` as `payload` instead of `payload.payload`
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
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.
- 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`
- Fixed in Releases 2.0.0 added
- Status changed from Ready For Testing to Closed
Also available in: Atom
PDF
Updated by The Foreman Bot over 5 years ago
Updated by Tomer Brisker over 5 years ago
Updated by