-
Notifications
You must be signed in to change notification settings - Fork 0
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
chore: added title to Veolas for duplicate title error in SEMrush #135
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
4 Skipped Deployments
|
import { IncreaseLockModal } from "./IncreaseLockModal"; | ||
import { VeOlasManage } from "./VeOlasManage"; | ||
import { GetServerSideProps } from "next"; | ||
import Head from "next/head"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
your linter is not working 🥲
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- need to fix linter, github actions are failed
- don't understand what it means "duplicated titles" in SEMrush, currently there's only one on veOLAS page 🤔
- we didn't fix it for the contracts page, is that expected?
Lock OLAS for veOLAS | ||
</Button> | ||
<> | ||
<Head> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you please add the whole meta block to govern app similar to one that we have on olas website or contribute or in bond app here apps/bond/pages/_app.jsx? currently govern app is not searchable in google :( I probably forgot to add that when created the app
</Button> | ||
<> | ||
<Head> | ||
<title>VeOlas</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be veOLAS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or maybe Govern | veOLAS
apps/govern/pages/veolas.tsx
Outdated
const VeOlasPage = () => ( | ||
<> | ||
<Meta | ||
pageTitle="veOlas" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pageTitle="veOlas" | |
pageTitle="veOLAS" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a small comment, approving in advance, otherwise looks very good! thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice work 👏
Proposed changes
Just added a title to VeOlas to resolve SEMrush issues, duplicate title tags: