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

Add job page (#87) #110

Merged
merged 2 commits into from
Jul 28, 2020
Merged

Add job page (#87) #110

merged 2 commits into from
Jul 28, 2020

Conversation

baijunyao
Copy link
Member

请保留以下声明。

我已阅读投稿指南并授权云原生社区发布我原创或翻译的文章,我将保留对本文署名的权利。

@netlify
Copy link

netlify bot commented Jul 27, 2020

Deploy preview for cloudnativeto ready!

Built with commit 3ae2ac1

https://deploy-preview-110--cloudnativeto.netlify.app

@baijunyao
Copy link
Member Author

@rootsongjc 直接把你博客的页面抄了过来;保留了一篇招聘作为示例;

@rootsongjc
Copy link
Member

rootsongjc commented Jul 27, 2020

从预览效果来看 https://deploy-preview-110--cloudnativeto.netlify.app/job/intern-tnt/ 招聘的详情页面上少了一个标题。
我觉得可以在 Markdown 的 header 里加一些配置,比如:

  • 公司
  • 部门
  • 岗位
  • base 地
  • 类型(实习生、社招)
  • 部门

确定固定格式,要求所有提交招聘需求的人都要按照该格式填写。

@baijunyao
Copy link
Member Author

标题加上了;

@rootsongjc
Copy link
Member

image
这个字体换成白色的,现在的灰色看不清楚。

@baijunyao
Copy link
Member Author

改了;

Copy link
Member

@rootsongjc rootsongjc left a comment

Choose a reason for hiding this comment

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

/LGTM

@mergify mergify bot merged commit 6a02899 into cloudnativeto:master Jul 28, 2020
@rootsongjc rootsongjc mentioned this pull request Jul 28, 2020
@@ -6,8 +6,11 @@
<body>
{{- partial "preloader.html" . -}}
{{- partial "header.html" . -}}
{{ if not .IsHome }}
Copy link
Member

@rootsongjc rootsongjc Jul 28, 2020

Choose a reason for hiding this comment

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

这里加了这个之后导致了所有页面的 page header 都多了一个,如下图。
image

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.

None yet

5 participants