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

DyUser.java:44-50: At the moment, the repos()... #409

Closed
davvd opened this issue Oct 31, 2014 · 30 comments
Closed

DyUser.java:44-50: At the moment, the repos()... #409

davvd opened this issue Oct 31, 2014 · 30 comments

Comments

@davvd
Copy link
Collaborator

davvd commented Oct 31, 2014

Puzzle 374-bb504035 in src/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

@davvd
Copy link
Collaborator Author

davvd commented Oct 31, 2014

@soliveri This task is yours, please go ahead keeping in mind this. If any questions, don't hesitate to ask right here

This task's budget is 30 mins. This is exactly how much will be paid when the problem explained above is solved. See this for more information

@davvd
Copy link
Collaborator Author

davvd commented Nov 11, 2014

@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

@davvd
Copy link
Collaborator Author

davvd commented Nov 19, 2014

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

@davvd
Copy link
Collaborator Author

davvd commented Nov 21, 2014

@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

@davvd davvd removed the @soliveri label Nov 21, 2014
@davvd
Copy link
Collaborator Author

davvd commented Nov 21, 2014

@malafeev This task is yours, please go ahead keeping in mind this. If any questions, don't hesitate to ask right here

The budget of this issue is 30 mins, which is exactly how much will be paid when the task is done (see this for explanation)

@davvd
Copy link
Collaborator Author

davvd commented Dec 2, 2014

@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

@davvd
Copy link
Collaborator Author

davvd commented Dec 12, 2014

@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

@davvd davvd removed the @malafeev label Dec 12, 2014
@davvd
Copy link
Collaborator Author

davvd commented Dec 12, 2014

@caarlos0 the task is yours please proceed

@davvd
Copy link
Collaborator Author

davvd commented Dec 23, 2014

@caarlos0 you're working with this issue for 10 days already

@caarlos0
Copy link
Contributor

@davvd sorry, I'm on vacations, please assign to someone else.

@davvd
Copy link
Collaborator Author

davvd commented Dec 24, 2014

@davvd sorry, I'm on vacations, please assign to someone else.

@caarlos0 right, I will find someone else, no problem

@davvd davvd removed the @caarlos0 label Dec 24, 2014
@davvd
Copy link
Collaborator Author

davvd commented Dec 24, 2014

@cyberone please go ahead, it's your task now, keep this in mind, and don't hesitate to ask any technical questions you may have

The cost of this task is 30 mins (this is exactly how much will be paid, not less not more), when the task is done

@davvd
Copy link
Collaborator Author

davvd commented Jan 1, 2015

@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

@davvd
Copy link
Collaborator Author

davvd commented Jan 7, 2015

@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

@davvd davvd removed the @cyberone label Jan 7, 2015
@davvd
Copy link
Collaborator Author

davvd commented Jan 7, 2015

@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)

@davvd davvd added the @wb14123 label Jan 7, 2015
@wb14123
Copy link
Contributor

wb14123 commented Jan 14, 2015

If DyRepos include a user criteria, how to construct it in DyBase.repos()? In my understand, DyBase.repos() should return all users' repos. I think I could return all repos in DyRepos while user criteria is null, is that OK?

@yegor256
Copy link
Owner

@wb14123 we never use NULL anywhere, see http://www.yegor256.com/2014/05/13/why-null-is-bad.html

@davvd
Copy link
Collaborator Author

davvd commented Jan 19, 2015

@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

@davvd
Copy link
Collaborator Author

davvd commented Jan 27, 2015

@wb14123 the task is overdue, and I have to re-assign it to someone else. Please, stop working with it immediately. In general, we're against overdue tasks, check this page. added -60 to your rating, now it is equal to -75

@davvd davvd removed the @wb14123 label Jan 28, 2015
@davvd
Copy link
Collaborator Author

davvd commented Feb 25, 2015

@carlosmiranda it's yours,please go ahead

@carlosmiranda
Copy link
Contributor

@davvd We're waiting for #524.

@davvd
Copy link
Collaborator Author

davvd commented Mar 11, 2015

@davvd We're waiting for #524.

@carlosmiranda yep, got it, let's wait for #524 being solved

@davvd
Copy link
Collaborator Author

davvd commented Mar 18, 2015

we were waiting for #524 - it is closed already

carlosmiranda added a commit to carlosmiranda/thindeck that referenced this issue Mar 19, 2015
carlosmiranda added a commit to carlosmiranda/thindeck that referenced this issue Mar 19, 2015
carlosmiranda added a commit to carlosmiranda/thindeck that referenced this issue Mar 19, 2015
@carlosmiranda
Copy link
Contributor

@davvd we're waiting for #528 now (pull request).

@davvd
Copy link
Collaborator Author

davvd commented Mar 19, 2015

@davvd we're waiting for #528 now (pull request).

@carlosmiranda OK, let's wait for #528

carlosmiranda added a commit to carlosmiranda/thindeck that referenced this issue Mar 19, 2015
carlosmiranda added a commit to carlosmiranda/thindeck that referenced this issue Mar 19, 2015
@davvd
Copy link
Collaborator Author

davvd commented Mar 20, 2015

we were waiting for #528 - it is closed already

@davvd
Copy link
Collaborator Author

davvd commented Mar 23, 2015

there is a puzzle in this code 409-272f0225, we'll resolve it later

@davvd
Copy link
Collaborator Author

davvd commented Mar 24, 2015

Finished, thanks, closing..

@davvd davvd closed this as completed Mar 24, 2015
@davvd
Copy link
Collaborator Author

davvd commented Mar 24, 2015

@carlosmiranda 30 mins added to your account (payment number 54118997), many thanks for your contribution!... +30 added to your rating, at the moment it is: +4096

@davvd
Copy link
Collaborator Author

davvd commented Jul 3, 2015

@yegor256 the last puzzle 409-272f0225/#537 solved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants