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

Standardised AWS before NCI access link #301

Merged
merged 2 commits into from
Sep 6, 2024

Conversation

benji-glitsos-ga
Copy link
Collaborator

For consistency, ordered the AWS access link before the NCI one on all active product pages.

Copy link

github-actions bot commented Aug 30, 2024

Preview is ready!

View the preview: 🌐 PR Preview 301

You can also view the deploy log or read the documentation.

@benji-glitsos-ga benji-glitsos-ga changed the title Standardised AWS before NCI in all access links Standardised AWS before NCI access link Aug 30, 2024
Copy link
Collaborator

@CEKrause CEKrause left a comment

Choose a reason for hiding this comment

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

Something was going a bit funny with the preview (I was getting 404s), but the code changes look fine, so approved!

@benji-glitsos-ga benji-glitsos-ga merged commit 8917d5a into main Sep 6, 2024
1 check passed
@benji-glitsos-ga benji-glitsos-ga deleted the feature/standardised-aws-before-nci branch September 6, 2024 00:32
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