Skip to content

Website updates - #49

Merged
demiankatz merged 12 commits into
UniversalViewer:astro2026from
Saira-A:website-updates
Jun 24, 2026
Merged

Website updates#49
demiankatz merged 12 commits into
UniversalViewer:astro2026from
Saira-A:website-updates

Conversation

@Saira-A

@Saira-A Saira-A commented Jun 18, 2026

Copy link
Copy Markdown
Contributor
  • Moves social media statement from contact page to policies page
  • Updates the National Library of Scotland's logo to their new one
  • Removes Google Calendar from community page
  • Adds social media logos to contact page
  • Adds text to policies page
  • Updates roadmap page with coming soon text
  • Moves showcase above features on index page
  • Removes "logo" text from footer

@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

@Saira-A is attempting to deploy a commit to the Universal Viewer Team on Vercel.

A member of the Team first needs to authorize it.

@Saira-A Saira-A mentioned this pull request Jun 18, 2026

@demiankatz demiankatz left a comment

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.

Thanks, @Saira-A! See below for a couple of questions.

Comment thread src/pages/policy.astro Outdated
Comment thread src/pages/policy.astro Outdated
@Saira-A

Saira-A commented Jun 19, 2026

Copy link
Copy Markdown
Contributor Author

Added the links, thanks @demiankatz

@demiankatz demiankatz left a comment

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.

Thanks, @Saira-A, looks good to me now. Is this ready to merge, or do you plan to do anything more?

@erinburnand

Copy link
Copy Markdown

There's a couple of tasks on #165 that I didnt notice - should we at least make the edits to the documentation and events page (if we haven't already?)

@Saira-A

Saira-A commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

That's all for now, thanks @demiankatz @erinburnand

@Saira-A
Saira-A requested a review from demiankatz June 24, 2026 16:01

@demiankatz demiankatz left a comment

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.

Changes look reasonable to me based on reviewing the code here. I wanted to look at this in context, but the Vercel preview seems to be broken (and I just get an error message when I try to approve the deployment).

I'm approving so you can merge if you want -- not sure if it's worth trying to troubleshoot Vercel or do a local build first, but I'm short on time today and don't want to hold you up.

@Saira-A

Saira-A commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

I don't actually have permission to merge but it's not urgent so can wait until you have more time :)

@demiankatz

Copy link
Copy Markdown
Contributor

Had time this afternoon to spin up my VM and look at everything, and I see no problems. Thanks again, @Saira-A!

@demiankatz
demiankatz merged commit 8646645 into UniversalViewer:astro2026 Jun 24, 2026
1 check failed
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