Uploaded image for project: 'Satellite'
  1. Satellite
  2. SAT-14893 Add validations for Simplified ACS update via hammer
  3. SAT-15228

[DEV] Add validations for Simplified ACS update via hammer

XMLWordPrintable

    • Sprint 111, Sprint 112, Sprint 113

      We discovered that the params are being silently filtered out, which is causing the lack of error. One idea is to throw validation errors in the controller. However, it seems the "Rails way" is to keep the controller light and rely on model validations. Model validations are conditional and have custom messages, so perhaps we can permit all ACS potential params and remove the conditional param filtering.

       

      DEV Tracker for https://issues.redhat.com/browse/SAT-14893
      Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2160112

              rhn-engineering-qjames Quinn James
              satellite-focaccia-bot Focaccia Bot
              Jeremy Lenz Jeremy Lenz
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Created:
                Updated:
                Resolved: