Skip to content
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

New documentation for document-collection emails #4631

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

hannako
Copy link
Contributor

@hannako hannako commented May 2, 2024

Add documentation to explain the taxonomy_topic_email_override feature present on some document collection pages.

@hannako hannako force-pushed the override_email_notifications branch 6 times, most recently from ecf24d6 to 66e055f Compare May 2, 2024 21:50
Copy link
Contributor

@leenagupte leenagupte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, this is a really hard snowflake ❄️ to articulate, but I think you've done a really good job. 🎉

I've added some in-line comments where I think things might be unclear.

I half-remember what went on and what was agreed, so I could fill in some, but not all of of the blanks. However, I think though, no knowledge should be assumed, and these docs should be as hand-holdy as possible because as you know emails in general, and this in particular, are very complex concepts to get your head around.


#### How to set a taxonomy topic email override field for previously published pages

It is not possible to set this field on a previously published page. If a request is made to `2nd line - GOV.UK content` to add this feature to a live page, their documentation aligns with the guidance below:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you trying to say that the 2nd line - GOV.UK content guidance is the same as the bullet points below?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have amended

Comment on lines 34 to 40
#### How to check if a document_collection has previously been published

1. Find the document_collection in the whitehall admin UI
2. Find the ID. E.g. `12345` for a document_collection at URL `whitehall-admin.publishing.service.gov.uk/government/admin/collections/12345`
3. `DocumentCollection.find(12345).document.live?` returns `true` for previously published document_collections.

#### How to set a taxonomy topic email override field for new pages
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe "finding the document collection id" is a necessary step for both "checking if the document collection has been published" and to "set the taxonomy topic override".

However the why the steps are laid out makes it look you always need to check if a document collection has been published first, but I believe the rake task checks this for you, and raises an exception.

Perhaps have the "How to check if a document_collection has previously been published" steps ask people to run the rake task in "dry run" mode to check for errors rather than spinning up a rails console?

I'd also duplicate the step for finding the document collection id before the instructions for running the rake task.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah think I've not addressed the last sentence here so I won't resolve this comment!

@hannako
Copy link
Contributor Author

hannako commented May 8, 2024

Thanks @leenagupte - Much appreciated!

@hannako hannako force-pushed the override_email_notifications branch from 66e055f to 3bfe3f3 Compare May 29, 2024 23:07
@hannako hannako force-pushed the override_email_notifications branch from 3bfe3f3 to 94bea3c Compare May 29, 2024 23:21
@hannako hannako force-pushed the override_email_notifications branch from 94bea3c to 0f48cb2 Compare May 29, 2024 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants