Fix Content API Trash using new DJES delete-index-on-save

theonion / django-bulbs

DEPRECATED: This project is now part of the Mono Repo (https://github.com/theonion/omni)

MIT License

26 stars 7 forks source link

Fix Content API Trash using new DJES delete-index-on-save #167

Closed mparent61 closed 8 years ago

mparent61 commented 8 years ago

Paired with https://github.com/theonion/djes/pull/29

Originally was treating ES 404 error as failure. There's no reason to error out if what you're trying to delete is already absent.

@benghaziboy @MichaelButkovic

Will need to coordinate versioning w/ @kand

camsom commented 8 years ago

🔔

camsom commented 8 years ago

lol 👍

coveralls commented 8 years ago

Coverage decreased (-0.02%) to 86.272% when pulling 5cf2e21efb0c0e507ac5060efbf08fffb5143368 on fix-trash-es-index-delete into 68714391f8eb1ae72fe5739b706efc8b6a49b795 on master.

coveralls commented 8 years ago

Coverage decreased (-0.02%) to 86.272% when pulling 29ad1d034938ffca76470194ea38db9889ce00a6 on fix-trash-es-index-delete into 68714391f8eb1ae72fe5739b706efc8b6a49b795 on master.