Skip to content
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

feat: update cp failed task key #134

Merged
merged 3 commits into from
Nov 22, 2022
Merged

Conversation

sergebouchut2
Copy link
Contributor

@sergebouchut2 sergebouchut2 commented Oct 7, 2022

Signed-off-by: Serge Bouchut [email protected]

API has changed: we now have a flat field failed_task_key (as failed_task.category has been removed).
See: Substra/substra-backend#525

Linked to this [ASANA TASK](link here)

Description

How to test

Screenshots

Notes for developers and reviewers:

  • Think to update CHANGELOG.md before merge if needed !

@sergebouchut2 sergebouchut2 changed the title feat: remove failed task category feat: update failed task key Oct 7, 2022
@sergebouchut2 sergebouchut2 force-pushed the feat/remove_failed_task_category branch from 30e8ba8 to eb1d757 Compare October 7, 2022 12:35
@sergebouchut2 sergebouchut2 changed the title feat: update failed task key feat: update cp failed task key Oct 7, 2022
CHANGELOG.md Outdated
@@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Open task drawer directly in cp details page (#122)
- Algo creation events aren't included in newsfeed anymore (#127)
- Renamed any tuple thing into a task thing (#129)
- Update compute plan failed task key
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you include the PR number here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done 23ad981

@Milouu Milouu force-pushed the feat/remove_failed_task_category branch from 23ad981 to 248ff37 Compare November 22, 2022 16:52
@Milouu Milouu merged commit 3130483 into main Nov 22, 2022
@Milouu Milouu deleted the feat/remove_failed_task_category branch November 22, 2022 16:55
@Milouu Milouu mentioned this pull request Dec 19, 2022
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.

3 participants