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

Initial guidance page drafts #1339

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

Initial guidance page drafts #1339

wants to merge 4 commits into from

Conversation

mark-avery-dfe
Copy link
Collaborator

Context

Ticket: https://dfedigital.atlassian.net/browse/CPDNPQ-000

[Why are we making this change?]

Changes proposed in this pull request

[What has been changed?]

Failed feature specs screenshots

If any of the feature specs would fail, there will be page on the wiki with all
failures:

https://github.com/DFE-Digital/npq-registration/wiki/

@mark-avery-dfe mark-avery-dfe requested a review from a team as a code owner April 30, 2024 13:47
Copy link


### How to use an authentication token

Include an authentication token in all requests to the API by adding an <code>Authorization</code> request header (not as part of the URL) in the following format:
Copy link
Member

Choose a reason for hiding this comment

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

You should be able to use backticks here instead of <code> tags.

Suggested change
Include an authentication token in all requests to the API by adding an <code>Authorization</code> request header (not as part of the URL) in the following format:
Include an authentication token in all requests to the API by adding an `Authorization` request header (not as part of the URL) in the following format:

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It seems to depend on the context. For example, within tables the backticks don't work...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants