Update PUT /ad/{id} to include too many ads in the category error

ecg-icas / icas

iCAS documentation and changelogs

4 stars 2 forks source link

Update PUT /ad/{id} to include too many ads in the category error #93

Closed SithEngineer closed 1 year ago

twessling-icas commented 1 year ago

This should not be an error imo; we also allow POST /ad in a full category, the ad will just get paused. I suggest we do the same with the PUT /ad/{id}?

twessling-icas commented 1 year ago

As discussed, we go ahead with this for now. We will put a task on our backlog to make insert & update more uniform, changing the status of the ad to PAUSED if the user has reached the limit of active ads in the category.