-
Notifications
You must be signed in to change notification settings - Fork 121
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
3 changed files
with
3 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,3 @@ | ||
module Nesta | ||
VERSION = '0.17.0' | ||
VERSION = '0.18.0' | ||
end |
5d583e8
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.
Not sure where to put these comments, but wanted to alert you to two issues that came up for me, upgrading to this version:
2.4.0
but not2.5.0
. If I use2.5.0
I get the erroruninitialized constant Tilt::MarukuTemplate (NameError)
set :host_authorization, { permitted_hosts: [] }
to be added toapp.rb
in my themes. If I do not do this, I get aHost not permitted
error when I try to access the site (this doesn't happen running locally; only when deployed). See here for the same error in a different context. I am sure there is a more elegant way to solve the problem, and perhaps it should be solved inside Nesta itself rather than in the themes?Happy to raise these as issues if you think some modification of Nesta is needed; otherwise, just FYI!
5d583e8
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.
@etc Hi Brad. It looks like Tilt 2.5.0 has dropped support for Maruku. You can continue to use 2.4.0 though, Nesta doesn't (currently) have an opinion on which 2.x version of Tilt you use, so long as it's at least 2.1. That situation will only change if Tilt introduces breaking changes, and while I suspect it'll continue to evolve, it's been fairly slow so far. So you can pin the
tilt
gem to version 2.4.0 in yourGemfile
if you like.Also, see this note on the future of Maruku.
Filing an issue for the second problem would be appreciated, cheers.
5d583e8
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.
Hi Graham,
Thanks!
tilt
to 2.4.0 is what I have done. Is there a reason for keeping Maruku support enabled in Nesta, so that this is required? Personally I'd love to phase that out and bring Pandoc support in, which I would consider moving to from Kramdown.5d583e8
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.
Ah, good question. I did think that I should check whether it was required by default! I'll remove it.
Cheers for filing the issue.
On pandoc support, I wonder whether a plugin (or if it's really simple, a snippet of code in a site's
app.rb
) might do it. I've not looked into what'd be required, but as a lot of these rendering libraries have become unmaintained and/or been removed from Tilt, it's required a reasonable amount of updates to the main project.I've never used pandoc though, I should probably take a look and see what I think. I see that Tilt supports it for converting Markdown β is using it through Tilt what you had in mind?