Skip to content

Update Attend Page#19

Merged
exoticDFT merged 5 commits into
USRSE:mainfrom
suzanneprentice:suzanneprentice-attend-page
May 29, 2026
Merged

Update Attend Page#19
exoticDFT merged 5 commits into
USRSE:mainfrom
suzanneprentice:suzanneprentice-attend-page

Conversation

@suzanneprentice

Copy link
Copy Markdown
Contributor

Description

Checklist:

  • (Committee Chairs Only) I have posted the link for the PR to ask for content reviewers
  • I have previewed changes locally
  • I have updated the README.md if necessary

cc ADD GITHUB HANDLES HERE

Comment thread pages/attend/overview.md

@exoticDFT exoticDFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good overall! Just a couple of minor changes and some general comments.

Comment thread pages/attend/travel.md Outdated
Comment thread pages/attend/overview.md
Comment thread pages/attend/travel.md
Comment thread _data/navigation.yml
Comment thread pages/attend/overview.md Outdated

@exoticDFT exoticDFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great! I'm going to add one more modification to this which is removing the "Review" box on the front page. I'll work on that now. If @suzanneprentice you want to take a super quick look after I make that addition, we can hopefully then just merge the PR.

Comment thread pages/attend/travel.md
@exoticDFT

Copy link
Copy Markdown
Member

Looks great! I'm going to add one more modification to this which is removing the "Review" box on the front page. I'll work on that now. If @suzanneprentice you want to take a super quick look after I make that addition, we can hopefully then just merge the PR.

Ahh shoot I can't edit this directly since this is coming from your fork. Would you be able to add that one modification? So, that we aren't leaving this lingering too long, I'll just go ahead and merge this in ~an hour and create another PR for the removal later.

@exoticDFT exoticDFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks wonderful! Thanks @suzanneprentice!

@exoticDFT exoticDFT merged commit 8c02c18 into USRSE:main May 29, 2026
2 checks passed
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