-- Schema (migration 005_fieldforce_phase2.sql)
CREATE TABLE org_module_configs (
  id         uuid PRIMARY KEY DEFAULT gen_random_uuid(),
  org_id     text NOT NULL,
  module     text NOT NULL,
  config     JSONB NOT NULL DEFAULT '{}',
  created_at timestamptz NOT NULL DEFAULT now(),
  updated_at timestamptz NOT NULL DEFAULT now()
);
CREATE UNIQUE INDEX ON org_module_configs (org_id, module);

GET /api/organizations/:org_id/configs/fieldforce
  1. SELECT config FROM org_module_configs WHERE org_id = ? AND module = 'fieldforce'
  2. Row found → unmarshal config JSONB → merge missing fields from DefaultFieldforceConfig
  3. Row not found → return DefaultFieldforceConfig unchanged

PUT /api/organizations/:org_id/configs/fieldforce
  Body: { "overdue_notify_after_hours": 2 }
  → Validate: no negative values
  → INSERT INTO org_module_configs ... ON CONFLICT (org_id, module) DO UPDATE SET config = EXCLUDED.config
  → Return merged config (stored + defaults for unset fields)

var DefaultFieldforceConfig = FieldforceConfig{
    OverdueNotifyAfterHours: 0,
    EscalationAfterHours:    24,
}

// Creating a per-module table — defeats the purpose of the unified table
// migration: CREATE TABLE org_fieldforce_configs (org_id, overdue_hours, ...)  ← wrong

// Returning raw JSONB without merging defaults
func GetFieldforceConfig(orgID string) map[string]any {
    var row OrgModuleConfig
    db.Where("org_id = ? AND module = 'fieldforce'", orgID).First(&row)
    return row.Config // wrong — unset fields return zero values, not defaults
}

// Allowing any role to update config
func (h *ConfigHandler) UpdateConfig(c echo.Context) error {
    // missing role check — must require Admin/Owner
    h.service.Set(ctx, orgID, module, config)
}

// Merge stored config with hardcoded defaults
func (s *OrgModuleConfigService) GetFieldforceConfig(ctx context.Context, orgID string) (FieldforceConfig, error) {
    cfg := DefaultFieldforceConfig // start with defaults
    var row OrgModuleConfig
    err := s.repo.FindByOrgAndModule(ctx, orgID, "fieldforce", &row)
    if errors.Is(err, gorm.ErrRecordNotFound) {
        return cfg, nil // no row → full defaults
    }
    if err != nil { return cfg, err }
    // merge: only override defaults for fields explicitly set in stored config
    if v, ok := row.Config["overdue_notify_after_hours"]; ok {
        cfg.OverdueNotifyAfterHours = int(v.(float64))
    }
    if v, ok := row.Config["escalation_after_hours"]; ok {
        cfg.EscalationAfterHours = int(v.(float64))
    }
    return cfg, nil
}