cds-snc / platform-forms-client

NextJS application that serves the public-facing website for Forms
https://forms-staging.cdssandbox.xyz/
MIT License
31 stars 12 forks source link

Section title is not behind conditional logic feature flag #3755

Closed anikbrazeau closed 3 weeks ago

anikbrazeau commented 4 weeks ago

Description

Section title for the confirmation message is a new concept that has not yet been introduced.

Steps to reproduce

  1. Go to staging
  2. Click on Edit
  3. Scroll down to confirmation message
  4. See Section title tag

Details

Expected behaviour

Only release mentions of sections when available?

Screenshots or videos

Screenshot 2024-06-06 at 11 09 58 AM

timarney commented 3 weeks ago

@Abi-Nada @samsadasivan

I can make a quick PR on Monday to fix this before release

Should we just hide the "Section title" badge or do we need to do anything else?

Abi-Nada commented 3 weeks ago

@timarney I think it should be fine if we just hide the Section Title, but deferring to @samsadasivan on any design input

timarney commented 3 weeks ago

PR is ready to go

https://github.com/cds-snc/platform-forms-client/pull/3769

Screenshot 2024-06-10 at 7 48 46 AM