Skip to content

Add sorting by highest price and number of offers#299

Merged
vohmar merged 12 commits intomasterfrom
add-sorting-by-highest-price-and-number-of-offers
Jul 25, 2019
Merged

Add sorting by highest price and number of offers#299
vohmar merged 12 commits intomasterfrom
add-sorting-by-highest-price-and-number-of-offers

Conversation

@maciej-szlosarczyk
Copy link
Contributor

@maciej-szlosarczyk maciej-szlosarczyk commented Jul 8, 2019

In administrator UI, you can now sort auctions by highest offer and number of offers.A participant can also order auctions by a new parameter - his/her own biggest offer. Both also work in search.

Fxes #128, should improve overall performance.

Things to check while testing:

  • JSON API continues to return all active auctions, even if there's more than 20 (that's per page limit)
  • No information leaks from auction index page. Particularly, no user can get different's user offers.

@maciej-szlosarczyk maciej-szlosarczyk force-pushed the add-sorting-by-highest-price-and-number-of-offers branch from 406c78b to deb59a0 Compare July 8, 2019 09:29
@maciej-szlosarczyk maciej-szlosarczyk force-pushed the add-sorting-by-highest-price-and-number-of-offers branch from 4e2ced1 to 7c86bc9 Compare July 11, 2019 06:23
@vohmar vohmar assigned maciej-szlosarczyk and unassigned vohmar Jul 22, 2019
@maciej-szlosarczyk maciej-szlosarczyk force-pushed the add-sorting-by-highest-price-and-number-of-offers branch from 7c86bc9 to e45fb51 Compare July 22, 2019 10:26
@vohmar vohmar assigned maciej-szlosarczyk and unassigned vohmar Jul 23, 2019
@maciej-szlosarczyk maciej-szlosarczyk force-pushed the add-sorting-by-highest-price-and-number-of-offers branch from 4e7bbc3 to cd5fa8b Compare July 23, 2019 11:45
@vohmar vohmar merged commit 091d850 into master Jul 25, 2019
@vohmar vohmar deleted the add-sorting-by-highest-price-and-number-of-offers branch July 30, 2019 06:42
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