-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
StoredProjectManagerTestCase.java: Unignore/update this test onc... #1271
Comments
@amihaiemil this is your task now, please go ahead. Deadline (when this ticket should be closed) is Estimation here is Remember, you don't have to solve everything in this ticket - you can solve it partially and leave If you have any questions don't ask me, I'm not a technical person. Open new tickets instead. If you don't have time or simply don't want to work on this, you can always resign. |
@zoeself resign |
@amihaiemil ok, I'll find someone else to take care of it. |
@criske this is your task now, please go ahead. Deadline (when this ticket should be closed) is Estimation here is Remember, you don't have to solve everything in this ticket - you can solve it partially and leave If you have any questions don't ask me, I'm not a technical person. Open new tickets instead. If you don't have time or simply don't want to work on this, you can always resign. |
@criske Don't forget to close this ticket before the deadline (2021-11-13T09:31:45). You are past the first half of the allowed period. |
This logic will be tested along with self-xdsd#1271.
@zoeself remove |
@amihaiemil ok, I've removed this task from scope. I'm not managing it anymore. |
@zoeself add |
@amihaiemil I've just registered this ticket as a task and will assign it to someone soon. Thanks! |
@criske when sending the 'Estimation updated' comment, can we make it more detailed please? :D Something like: "Estimation was changed from x min to y min at <>". Timestamp is needed because the bot will not post the same comment twice, normally, but in this case we want to allow duplicated comments in case the user changes his mind and goes back to the previous estimation or whatever : D |
@criske this is your task now, please go ahead. Deadline (when this ticket should be closed) is Estimation here is Remember, you don't have to solve everything in this ticket - you can solve it partially and leave If you have any questions don't ask me, I'm not a technical person. Open new tickets instead. If you don't have time or simply don't want to work on this, you can always resign. |
- Unignored test - Updated Step api to be able to receive data from the previous Step in chain.
- reverted changes to Step API
#1271 Reply with estimation changed value.
Puzzle disappeared from the code, that's why I closed this ticket. |
@criske thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard. |
The puzzle
904709704
originating from #1265 has to be resolved:self-core/self-core-impl/src/test/java/com/selfxdsd/core/managers/StoredProjectManagerTestCase.java
Lines 1832 to 1833 in 3edc272
"Unignore/update this test once the whole functionality is refactored to use Steps.".
The puzzle was created by amihaiemil at
2021-10-30 10:34:14 +0300
.Estimation is
60 minutes
.If you have any technical questions, don't ask me, I won't be able to help. Open new issues instead.
The text was updated successfully, but these errors were encountered: