uchicago-sg / caravel

A Re-Re-Refresh the UChicago Marketplace
https://marketplace.uchicago.edu
16 stars 3 forks source link

Expire listings after 30 days #222

Closed fatlotus closed 8 years ago

fatlotus commented 8 years ago

This one ended up being trickier than I had expected :(

Main changes:

  1. Listings are no longer bumped to the top when edited.
  2. Listings are still visible, but do not come up in searches, after 30 days.

    209 #119 #216

georgeteo commented 8 years ago
  1. Do we want a way of pushing listings to the top? ("Refresh listing")?
  2. Will this solve our datastore size issue? (Exponentially increasing amount of stuff that we're storing?)
fatlotus commented 8 years ago
  1. Probably :( 'Nother pull request?
  2. Yes. This is the main goal, actually :p
fatlotus commented 8 years ago

aaand there's a weird bug

https://caravel-code-reviews.appspot.com/92f96446-1a0f-415c-bdcf-6de60edc1c25

fatlotus commented 8 years ago

Weird bug is now fixed.

fatlotus commented 8 years ago

Weeee! Bump button added. #208