-
Notifications
You must be signed in to change notification settings - Fork 5
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
Seo #942
Seo #942
Changes from 6 commits
44e5eab
2f1749a
9e53d09
3da870f
d66d802
e1ef3c0
98932b7
1633d22
822e4c4
d4da9d0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
#Generated by Gradle | ||
saos.version=0.9.5 | ||
saos.version=0.9.6-SNAPSHOT |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
<?xml version="1.0" encoding="UTF-8"?> | ||
<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9"> | ||
<url> | ||
<loc>https://www.saos.org.pl/</loc> | ||
<lastmod>2015-09-30</lastmod> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. the loc should be configured, you'll have the same value for test and devel env There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We have to talk about it a bit. |
||
<changefreq>weekly</changefreq> | ||
</url> | ||
<url> | ||
<loc>https://www.saos.org.pl/search</loc> | ||
<lastmod>2015-09-30</lastmod> | ||
<changefreq>daily</changefreq> | ||
</url> | ||
<url> | ||
<loc>https://www.saos.org.pl/analysis</loc> | ||
<lastmod>2015-09-30</lastmod> | ||
<changefreq>weekly</changefreq> | ||
</url> | ||
<url> | ||
<loc>https://www.saos.org.pl/help</loc> | ||
<lastmod>2015-09-30</lastmod> | ||
<changefreq>daily</changefreq> | ||
</url> | ||
</urlset> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
<%@ page language="java" contentType="text/html; charset=UTF-8" pageEncoding="UTF-8"%> | ||
<%@ include file="/WEB-INF/view/common/taglibs.jsp" %> | ||
|
||
|
||
<meta name="keywords" content="<spring:message code='analysis.meta.keywords' />" /> | ||
<meta name="description" content="<spring:message code='analysis.meta.pageDescription'/>"/> | ||
|
||
|
||
<%-- Open graph ogp.me --%> | ||
<%@ include file="../common/openGraphMeta.jsp" %> | ||
<meta property="og:title" content="<spring:message code="pageTitle.analysis" /> - <spring:message code="saos.fullnameAndShortcut"/>"> | ||
<meta property="og:description" content="<spring:message code='analysis.meta.pageDescription'/>"> | ||
|
||
|
||
<title><spring:message code="pageTitle.analysis" /> - <spring:message code="saos.fullnameAndShortcut"/></title> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
<%@ page language="java" contentType="text/html; charset=UTF-8" pageEncoding="UTF-8"%> | ||
<%@ include file="/WEB-INF/view/common/taglibs.jsp" %> | ||
|
||
<c:set var="pageContextRequest" value="${pageContext.request}" /> | ||
<c:set var="requestURL">${pageContextRequest.requestURL}</c:set> | ||
<c:set var="requestURI" value="${pageContextRequest.requestURI}" /> | ||
|
||
<%--Base url --%> | ||
<c:set var="basePath" value="${fn:substring(requestURL, 0, fn:length(requestURL) - fn:length(requestURI))}${pageContextRequest.contextPath}" /> | ||
|
||
<%-- Open graph ogp.me --%> | ||
<meta property="og:site_name" content="Saos"> | ||
<meta property="og:type" content="object"> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should the og:type be 'website'? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 👌 |
||
<meta property="og:image" content="${basePath}/static/image/saosLogo.png"> | ||
<meta property="og:url" content="${basePath}${requestScope['javax.servlet.forward.servlet_path']}"> | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You shouldn't take the request url like this. I told you earlier that you should use it HttpServletRequestUtils service to get the request url - we are behind proxy on prod and you have to take it into account. And you don't care about query parameters here, I'm not sure if it's ok. |
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 strange name of the key, I suggest details.meta.judges
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.
👌