Skip to content
This repository was archived by the owner on Jun 25, 2020. It is now read-only.

Conversation

@laedit
Copy link
Contributor

@laedit laedit commented Jan 4, 2015

This should fix #182.
The site.title has been implemented like jekyll, but the page.id has some differences:
In Jekyll with the permalink /:categories/:year/:month/:day/:title/index.html the page id will be something like that: "/fr/2014/11/26/myfirstpost/myfirstpost".
In Pretzel it will be: "/fr/2014/11/26/myfirstpost/".
The other difference is that in Jekyll the pages have no id, only the posts have. In Pretzel both will have a generated id.

@shiftkey, @JakeGinnivan are you okay with that?

Jérémie Bertrand added 4 commits December 29, 2014 19:25
Now the title can be defined in _config.yml and used in the liquid
template.
Now, the priority are:
- page.title in the page yaml front matter
- site.title in _config.yml
- SiteContext.Title

And add a few tests.
And the id can be defined in the page without any error but it will not
be used.
@shiftkey
Copy link
Member

shiftkey commented Jan 4, 2015

👍

@laedit
Copy link
Contributor Author

laedit commented Jan 7, 2015

@shiftkey sorry to bother you, but is the TeamCity server working?
If not can I set an AppVeyor.com project for the Ci part?

@shiftkey
Copy link
Member

shiftkey commented Jan 7, 2015

Not sure, I'd have to ask @JakeGinnivan what's up.

If you want to take charge of the CI process, I'd highly recommend AppVeyor. I don't think Jake would mind either!

@JakeGinnivan
Copy link
Member

Nope, happy for it to change.

I can also give you admin permission to the teamcity build if you want to modify that?

The biggest issue with the build is that when you release it doesn't upload the zip to github, it just pushes the chocolatey package which points to a zip which has not been uploaded

@JakeGinnivan
Copy link
Member

Build is working - http://teamcity.ginnivan.net/viewType.html?buildTypeId=bt52

Report status to github is not setup though.

@laedit
Copy link
Contributor Author

laedit commented Jan 7, 2015

I'll be happy to try to modify the teamcity build.

@JakeGinnivan
Copy link
Member

@laedit just create an account and I will set you up

@laedit
Copy link
Contributor Author

laedit commented Jan 7, 2015

Done. And sorry about the failing test, I will fix it ASAP.

@JakeGinnivan
Copy link
Member

@laedit done, you are project admin on the Pretzel project in teamcity.

If you need a hand setting anything up I am more than happy to help!

@laedit
Copy link
Contributor Author

laedit commented Jan 8, 2015

Weird, it seems that the GitHub report status plugin is well configured and activated but do nothing...
Could you update it to the latest version?
At least it supports the use of the GitHub personal access token, it will spare you to store you password.

@JakeGinnivan
Copy link
Member

Will try to update it tonight.

@laedit
Copy link
Contributor Author

laedit commented Jan 17, 2015

The Team CIty server seems to not responding. Not a big problem, I have set up an AppVeyor project.

@shiftkey there is one little problem thought, the AppVeyor do not seem to add a build in the queue for each push, could you tell me if the "push" and "pull request" trigger are checked in the repo options?
Thanks.

@laedit laedit merged commit af38066 into master Jan 17, 2015
@laedit laedit deleted the Fix-site.title-and-page.id branch January 17, 2015 15:50
@shiftkey
Copy link
Member

could you tell me if the "push" and "pull request" trigger are checked in the repo options?

I can only see "push" is configured - could you check the TeamCity settings and see if something is missing there?

@laedit
Copy link
Contributor Author

laedit commented Jan 18, 2015

Sorry, I wasn't clear, the Team CIty server is out.
But for AppVeyor, the configuration need to be done in the settings of the repo: could you check the "pull request" trigger for the AppVeyor webhook (snip)?
Thanks.

@shiftkey
Copy link
Member

@laedit ok, I've added the webhook for the AppVeyor URL

@laedit
Copy link
Contributor Author

laedit commented Jan 18, 2015

@shiftkey Thanks! 😄

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.

site.title is always null

4 participants