Skip to content

Updates the checkbox for multiple admin settings forms

Sam Beckham requested to merge admin-page-checkbox-round-one into master

What does this MR do and why?

Migrates the old, borwser based checkboxes on the admin setting page to use the new pyjamas one as part of Pajamas Migration Day

I batched a few of these since they're really small changes.

Screenshots or screen recordings

Before

Unchecked Checked
Screenshot_2022-04-08_at_16.42.29 Screenshot_2022-04-08_at_16.42.35
Screenshot_2022-04-08_at_16.42.46 Screenshot_2022-04-08_at_16.42.53
Screenshot_2022-04-09_at_11.09.08 Screenshot_2022-04-09_at_11.09.16
Screenshot_2022-04-08_at_16.43.11 Screenshot_2022-04-08_at_16.43.21
Screenshot_2022-04-08_at_16.43.31 Screenshot_2022-04-08_at_16.43.38
Screenshot_2022-04-08_at_16.43.44 Screenshot_2022-04-08_at_16.43.58
Screenshot_2022-04-08_at_16.44.07 Screenshot_2022-04-08_at_16.44.14

After

Unchecked Checked
Screenshot_2022-04-08_at_16.26.01 Screenshot_2022-04-08_at_16.26.07
Screenshot_2022-04-08_at_16.26.16 Screenshot_2022-04-08_at_16.26.25
Screenshot_2022-04-09_at_11.06.20 Screenshot_2022-04-09_at_11.09.44
Screenshot_2022-04-08_at_16.27.05 Screenshot_2022-04-08_at_16.27.11
Screenshot_2022-04-08_at_16.27.17 Screenshot_2022-04-08_at_16.27.28
Screenshot_2022-04-08_at_16.27.39 Screenshot_2022-04-08_at_16.27.45
Screenshot_2022-04-08_at_16.27.54 Screenshot_2022-04-08_at_16.28.02

How to set up and validate locally

  1. Go to the general admin settings GDK link
  2. Expand the relevant sections (see screenshots)
  3. Make sure the checkboxes work and toggle the correct settings.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

closes #358279 (closed) #358302 (closed) #358295 (closed) #358288 (closed) #358284 (closed) #358281 (closed) #358277 (closed) #358277 (closed)

Edited by Sam Beckham

Merge request reports