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

Exposes source_url and issues_url of cookbook #30

Merged
merged 1 commit into from
Dec 21, 2015

Conversation

ge1st
Copy link
Contributor

@ge1st ge1st commented Dec 21, 2015

Resolves #29

So Ridley is setting these as empty strings, thus the ugly empty calls.

ui
The cookbook detail now looks like ^^. Design guidance appreciated.

Anything I miss? Also didn't commit the minified css. Want me to?

@tapickell
Copy link
Contributor

@ge1st Looks clean to me, I like it. Go ahead and add minified css. Thank you for your help. 👍

@richardardrichard
Copy link
Contributor

It has been a while since I've dug around in here, but since everything is a static file, I think you would actually need to commit the minified css in order for any users to actually see the changes. There are tasks in the Rakefile to compile_css, and compile_js.

@tapickell
Copy link
Contributor

Created new issue about the process to minify CSS and JS files.
#31

@tapickell
Copy link
Contributor

Going to merge this without Min CSS b/c next PR will remove need to Min by hand. Thank you @ge1st

tapickell added a commit that referenced this pull request Dec 21, 2015
Exposes source_url and issues_url of cookbook
@tapickell tapickell merged commit 2a043d2 into electric-it:master Dec 21, 2015
@ge1st ge1st deleted the exposes-source-issues-url branch December 21, 2015 17:51
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.

3 participants