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

Should Unicorn Admin Theme be gem or just add to the cybele? #4

Closed
onurozgurozkan opened this issue Mar 14, 2013 · 9 comments
Closed
Labels

Comments

@onurozgurozkan
Copy link

@tayfunoziserikan
Copy link
Member

I think the add. 👍

@onurozgurozkan
Copy link
Author

Add to gem or just add to rails-template?

@tayfunoziserikan
Copy link
Member

Should be a Gem

@yunusozcan
Copy link

I think gem should be only asset pipeline for theme

2013/3/15 Tayfun Öziþ ERÝKAN [email protected]

Should be a Gem

Reply to this email directly or view it on GitHubhttps://github.com//issues/4#issuecomment-14954753
.

Yunus ÖZCAN
lab2023 - internet teknolojileri
Pamukkale Üniversitesi Teknoloji Geliþtirme Bölgesi B Blok Z13, Kýnýklý,
Denizli / Türkiye
http://lab2023.com
Tel: +90 258 215 5010 Fax: +90 212 381 6220

@yunusozcan
Copy link

maybe rails generator for layout & partials

@onurozgurozkan
Copy link
Author

If we just add the template not a gem, this repo should be private, but if we make sperate GEM. We can make this repo open source.

I think we should everything GEM. Because of standards of productivity. You can't modified the gem but you can modified the generators. It isn't good for standards.

Thats why 👍 for GEM.

@adamdilek
Copy link
Contributor

Unicorn should be gem. Our template default support unicorn layouts.

@tayfunoziserikan
Copy link
Member

Maybe we need to add a little rake task.

rake unicorn-theme:install
# layout is generated...OK
# controllers are generated...OK
# views are generated...OK
# partials are generated...OK
# etc. etc...

@onurozgurozkan
Copy link
Author

We will write our own rails admin template with bootstrap sass

https://github.com/lab2023/hierapolis

ismailakbudak pushed a commit that referenced this issue Jan 26, 2018
KBP-121 integrate responder

Approved-by: Tayfun Öziş ERİKAN <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants