Skip to content

Conversation

@hamirmahal
Copy link
Contributor

Describe this PR

Tab titles don't show | Prisma, but I think they should. If I favorite a Prisma docs page, I'd like to be able to find it easily by typing prisma into my browser's address bar, as opposed to trying to remember the specific title of the page I favorited.

Changes

This hardcodes homepage to false in src/templates/docs.tsx.

What issue does this fix?

This change fixes #5315.

@vercel
Copy link

vercel bot commented Sep 13, 2023

@hamirmahal is attempting to deploy a commit to the Prisma Team on Vercel.

A member of the Team first needs to authorize it.

@hamirmahal
Copy link
Contributor Author

Without this change

image

@hamirmahal
Copy link
Contributor Author

With this change

image

Docs homepage with this change

image

@ruheni
Copy link
Contributor

ruheni commented Sep 18, 2023

👋🏾 @hamirmahal

Thanks for looking into this. I've shared your PR internally for someone to review & share feedback on your PR. 🙂

@meletj meletj requested a review from nilubava September 18, 2023 14:46
Copy link
Contributor

@ruheni ruheni left a comment

Choose a reason for hiding this comment

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

Thanks again for the contribution 🙏

Copy link
Contributor

@nilubava nilubava left a comment

Choose a reason for hiding this comment

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

Thank you @hamirmahal

@vercel
Copy link

vercel bot commented Sep 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 18, 2023 3:13pm

@nilubava nilubava merged commit c0dc685 into prisma:main Sep 19, 2023
@hamirmahal hamirmahal deleted the fix/tab-titles-not-showing-prisma branch September 19, 2023 17:56
@hamirmahal
Copy link
Contributor Author

@nilubava You're welcome! Thanks for merging this.

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.

Tab titles don't include | Prisma

3 participants