Skip to content
This repository has been archived by the owner on Oct 16, 2024. It is now read-only.

Add /chat link #66

Merged
merged 2 commits into from
May 24, 2022
Merged

Add /chat link #66

merged 2 commits into from
May 24, 2022

Conversation

julieg18
Copy link
Contributor

@julieg18 julieg18 commented May 23, 2022

Fixes #41

@julieg18 julieg18 self-assigned this May 23, 2022
@gatsby-cloud
Copy link

gatsby-cloud bot commented May 23, 2022

Gatsby Cloud Build Report

mlem.ai

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 1m

Performance

Lighthouse report

Metric Score
Performance 💚 95
Accessibility 🔶 88
Best Practices 💚 100
SEO 🔶 83

🔗 View full report

titleTemplate: '%s | MLEM',
keywords: ['mlem']

const siteMetadata = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Went ahead and split up config for readability :)

}
]
},
'gatsby-plugin-meta-redirect'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

dvc.org and cml.dev redirects are set up with a nodejs server (since they're deployed with S3), but mlem.ai is just deployed with Heroku. So I used gatsby-plugin-meta-redirect. Are there any issues with using it? And for the future, do we want to deploy mlem.ai with S3?

src/components/Footer/index.tsx Outdated Show resolved Hide resolved
@julieg18 julieg18 requested a review from a team May 23, 2022 19:55
Copy link
Contributor

@rogermparent rogermparent left a comment

Choose a reason for hiding this comment

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

I'm glad it's this easy to use redirects on a more traditional Heroku SSG!

Copy link
Contributor

@yathomasi yathomasi left a comment

Choose a reason for hiding this comment

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

Nice work @julieg18.

@julieg18 julieg18 merged commit 7ada899 into main May 24, 2022
@julieg18 julieg18 deleted the add-chat-link branch May 24, 2022 15:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a mlem.ai/chat discord link
3 participants