Skip to content

#90 Adding identifiers for each headline in the checklist#154

Merged
sydseter merged 3 commits intomainfrom
identifiers
Aug 7, 2025
Merged

#90 Adding identifiers for each headline in the checklist#154
sydseter merged 3 commits intomainfrom
identifiers

Conversation

@sydseter
Copy link
Copy Markdown
Collaborator

@sydseter sydseter commented Aug 7, 2025

In this pull request:

  • This will add identifiers to each sub heading in the web checklist so that it is easier to track and make changes when items in the list need to be moved.

@sydseter sydseter requested a review from jgadsden August 7, 2025 10:59
and use the list below as suggestions for a checklist that has been tailored for the individual project.

#### 1. Implement Access Control
#### 1. Implement access control (AC)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is the only heading that is questionable - I agree that is should just be 'AC', do we want to change the heading to just 'Access Control' ?
In a way all the section headings could have 'Implement' at the beginning of the heading, so it would be good to remove the 'Implement' ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Agreed!

Copy link
Copy Markdown
Contributor

@jgadsden jgadsden left a comment

Choose a reason for hiding this comment

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

agree with all these changes @sydseter , just one comment on one of the headings

@sydseter sydseter requested a review from jgadsden August 7, 2025 11:09
@sydseter sydseter merged commit 9a3f157 into main Aug 7, 2025
@jgadsden jgadsden deleted the identifiers branch January 11, 2026 19:17
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.

2 participants