-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DSRC-65: Call to action block #1648
Conversation
Was there a requirement for the ability to have multiple CTA buttons next to one another? We also need a boolean field for the button which is We also need an optional |
Another thought - are these going into article pages or the yet-to-be-defined "standard page"? |
I just had a chat with Lizzy and she said they will be in a "standard" page, so I'll make a PR before this one is merged to add a standard page, add these components to that, and remove them from Article pages.
Content haven't asked for it... But I think there could be a case for that (at some point). We can always allow this to add multiple
👍
They have the |
Could we make it a list with a max of 1? I'm just thinking about the future - it would be more pain to change a dict to a list because the new format would "break" the frontend whereas adding more buttons, it might just not display them.
I was on about a specific |
Co-authored-by: Andrew Hosgood <andrew.hosgood@nationalarchives.gov.uk>
Ticket URL: DSRC-65 + DSRC-68
About these changes
Adds a CTA block (DSRC-65) and a button block (DSRC-68)
In the API:

In the CMS:

Before assigning to reviewer, please make sure you have
Merging PR guidance
Follow docs\developer-guide\contributing.md
Deployment guidance
Follow docs\infra\environments.md