search

onesteinbv / addons-project-nextcloud

1 stars 0 forks source link

Simplify "Nextcloud Event Status" #50

Open astirpe opened 1 year ago

astirpe commented 1 year ago

Model nc.event.status has only one field: name. I don't see any valid reason to have nc.event.status as a dedicated model for tracking the event status.

Proposal:

  1. remove nc.event.status model an the 3 values defined in nc_event_status_data.xml
  2. in calendar.event replace field nc_status_id of type Many2one with a field of type Selection, eg.:
nc_status = fields.Selection([("confirmed", "Confirmed"), ("tentative", "Tentative"), ("canceled", "Canceled")])