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

feat: prisma outlinks section #80

Merged
merged 1 commit into from
Sep 21, 2020
Merged

feat: prisma outlinks section #80

merged 1 commit into from
Sep 21, 2020

Conversation

nilubava
Copy link
Contributor

#55

Usage:

<PrismaOutlinks>
RELATED ON PRISMA.IO

- [Documentation - Advanced usage of generated types ](http://melconway.com/Home/Conways_Law.html)
- [Documentation - Installing the Prisma CLI](http://melconway.com/Home/Conways_Law.html)
- [Blog - Building a Modern Backend with TypeScript, PostgreSQL and Prisma: REST API, Validation, and Testing](http://melconway.com/Home/Conways_Law.html)

<withlogo>Prisma is an open-source database toolkit for Typescript and Node.js that aims to make app developers more productive and confident when working with databases.</withlogo>
</PrismaOutlinks>

If to be used in inner sections -

<PrismaOutlinks inner="true">
<withlogo>Prisma is an open-source database toolkit for Typescript and Node.js that aims to make app developers more productive and confident when working with databases.</withlogo>
</PrismaOutlinks>

@nilubava nilubava merged commit e8bc00c into master Sep 21, 2020
@nilubava nilubava deleted the issue-55 branch September 21, 2020 13:55
@hrescak
Copy link
Contributor

hrescak commented Sep 21, 2020

I feel like the RELATED ON PRISMA.IO and contents can be internal to the PrismaOutlinks components since they are not going to change across the pages?

@nilubava
Copy link
Contributor Author

@hrescak You mean the text 'RELATED ON PRISMA.IO' and the text after the Prisma Logo?

@hrescak
Copy link
Contributor

hrescak commented Sep 21, 2020

Yup! I think that doesn't change across different pages

@imchairmanm
Copy link
Contributor

@nilubava Would you like me to test this out as-is or should I wait for the changes you all have discussed?

This pull request was closed.
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