-
Notifications
You must be signed in to change notification settings - Fork 1
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: SEO improvements #4572
Comments
@techfg There are a number of SEO features currently supported.
However, we decided then to keep things simple for the time, and just implement a default robots.txt route, which gets you pretty far with SEO without having to define a full site map. What we implemented was this --- by default all Routes are blocked from crawlers, but any publicly-accessible Routes (Routes which your app's Public Profile can access) are marked as allowed, so that crawlers can index them. However, this is enabled on a per-Site basis, since, for example, you wouldn't want your QA/UAT site to be indexed by crawlers, generally you only want your Production site to be crawled. Thus, there is an "Enable SEO" checkbox on Sites, which turns on this behavior. |
Thanks @zachelrath, all great info!
I think expanding on this to support 301 makes sense. Possibly we minimize the administration of "redirect" routes so that instead of having to have a yaml for every route you want to redirect, we add the ability for glob patterns (or similar) using standard redirect syntax (e.g., think .htaccess type of thing). I can see a situation where users would move an entire set of pages (ues did so recently moving from /blog to /news) so supporting this type of thing en masse seems to make some sense. Thoughts?
Very cool, didn't realize adding to head like this was possible! Regarding some standard SEO related tags such as description, canonical, etc. possibly it makes sense to consider building in an automated "SEO" concept where if the view could contain an
Makes sense. We could just have an automated route when |
Would like to see the following added to the platform:
The text was updated successfully, but these errors were encountered: