-
Notifications
You must be signed in to change notification settings - Fork 6
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
Refactors lottery and lottery results #10
base: master
Are you sure you want to change the base?
Conversation
5fc9bba
to
d2c1968
Compare
d2c1968
to
d1d2c96
Compare
It also seems like these fields may be okay to remove from
|
Another idea... lottery information might fit better included as part of the |
60ee34c
to
b1b34e7
Compare
This API design has now been refactored:
Most of this information is already available on the
This also means that we can deprecate
This is the same as previously described.
Lottery Result is a resource within a listing that returns the information about a lottery number in a lottery. This includes
Web app can use this to look up both what preferences the applicant qualified for before the lottery and their preference rankings after the lottery. |
This PR represents 2 conceptual changes that 1) simplify and reduce duplication in the API and 2) meet the needs of new features for the web app.
Change 1:
/Listing/LotteryResult/{listingID} -> /Listing/{listingID}/Lottery
Lottery is a resource that related to a listing and holds information about the context of a lottery. This includes
Web app can use the
lotteryResults : boolean
property to determine whether or not there are resultsChange 2:
/Listing/LotteryResult/{listingID}/{lotteryNumber} -> /Listing/{listingID}/LotteryResult/{lotteryNumber}
Lottery Result is a resource within a listing that returns the information about a lottery number in a lottery. This includes
Web app can use this to look up both what preferences the applicant qualified for before the lottery and their ranking after the lottery.