-
Notifications
You must be signed in to change notification settings - Fork 20
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
DyUser.java:44-50: At the moment, the repos()... #409
Comments
@soliveri the task is your hands for the last 11 days.. keep in mind that if it's not closed in the next 48 hours, it will be re-assigned to someone else, see No Obligations principle |
The description of the task was changed (see above). If anybody has any questions or concerns about this change, please express them right now. Otherwise, please work with the new version |
@soliveri this task is taking too long, I have to change the performer, sorry. Please stop working with it right now. See our no obligations principle |
@malafeev you're holding this ticket for 10 days already. If it's not finished (and closed) in the next 48 hours, it will be re-assigned to someone else, see No Obligations principle |
@malafeev you've been working with this task for too long. I'm sorry, but I'll assign someone else. Please stop working with it. See our no obligations principle |
@caarlos0 the task is yours please proceed |
@caarlos0 you're working with this issue for 10 days already |
@davvd sorry, I'm on vacations, please assign to someone else. |
@cyberone check this "no obligations principle".. This task is on your name for at least 7 days. If you can't close it within the next 48 hours I'll have to assign someone else to it |
@cyberone this task is taking too long, I have to change the performer, sorry. Please stop working with it right now. See our no obligations principle -60 to your rating, your total score is -40 |
@wb14123 the task is for you now, follow these guidelines. Don't hesitate to ask any technical questions right here Total fixed cost of this task is 30 mins (see this for more info) |
If DyRepos include a user criteria, how to construct it in |
@wb14123 we never use NULL anywhere, see http://www.yegor256.com/2014/05/13/why-null-is-bad.html |
@wb14123 the ticket is with you for 11 days already. If it's not finished in the next 48 hours, it will be re-assigned to someone else, see No Obligations principle, this article should help if you're stuck; do not forget to remove the TODO puzzle from the source code; -30 to your rating, your total score is +45 |
@carlosmiranda it's yours,please go ahead |
@carlosmiranda yep, got it, let's wait for #524 being solved |
we were waiting for #524 - it is closed already |
@carlosmiranda OK, let's wait for #528 |
we were waiting for #528 - it is closed already |
there is a puzzle in this code |
Finished, thanks, closing.. |
@carlosmiranda 30 mins added to your account (payment number |
Puzzle
374-bb504035
insrc/main/java/com/thindeck/dynamo/DyUser.java:44-50
has to be resolved: At the moment, the repos() method returns all Repos associated with the Dynamo region, and this is definitely incorrect. I think we need to refactor DyRepos in order to include a user criteria (probably URN) in its constructor, and have DyUser pass it so that it will only return the associated criteria. I'm not completely sure about this design, feel free to implement something else if you think it's wrong. The intuition behind it is that we should only get the repos associated with the current user.If you have any technical questions, don't ask me, submit new tickets instead
The text was updated successfully, but these errors were encountered: