Skip to content

Conversation

@davehorsfall
Copy link
Collaborator

Description

  • Add Login Again link to logged out page
  • Add "Register" button to navigation (previously just login button)
  • Remove duplicated button in hero template
  • Add new image on homepage
image
  • Open intro video in modal
image
  • Update images on about page
image

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Technical work (non-breaking, change which is work as part of a new feature)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. python -m pytest)
  • The documentation builds and looks OK (eg. mkdocs serve)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future. (Indicate issue here: # (issue))

@codecov
Copy link

codecov bot commented Sep 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Contributor

@tdjames1 tdjames1 left a comment

Choose a reason for hiding this comment

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

This all looks great. Nice photos!

One more suggestion for the home page is that you could link to the "about" page from the final section ("A competencies framework for Digital Research Technical Professionals")

@davehorsfall
Copy link
Collaborator Author

Thansk @tdjames1. I've create a new Issue for your suggestion #497

@davehorsfall davehorsfall merged commit a6f2b6a into feat/conditional-user-landing Sep 7, 2025
5 checks passed
@davehorsfall davehorsfall deleted the feat/images-and-cleanups branch September 7, 2025 06:39
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.

3 participants