PatternBuilder / pattern-kit

A REST server for your Component API based Design System
20 stars 10 forks source link

Feature/jassmith/refactor and per module assets #20

Closed SiliconValet closed 5 years ago

SiliconValet commented 7 years ago
Refactor of pattern kit, additions to component renderer
* Factored methods out of large controllers.
* Removed redundant code
* Added a pattern model class to abstract the patterns a bit.
* Modified the way webcomponent renderer works.
* Added style component server.
* Changed from lzstring to pako for more portability across platforms (lzstring sucks at reproducibility cross-platform)
micahgodbolt commented 7 years ago

Yeah! Someone is finally cleaning up my crappy old code :D If you see something and think "wow thats a horrible hack!", yes it was. This was my first (and probably last) php project of this scale. Thanks for giving it some polish.

SiliconValet commented 7 years ago

I've got plenty of my own mess in there now, blame is slowly becoming mine.

Great concepts in there, works great as a dynamic templating engine. Most of what I've done with it is just housekeeping and stuff to extend the RESTful stuff. Thanks for the hard work!

micahgodbolt commented 7 years ago

I'm glad someone is enjoying and appreciating it! Have you heard about the movement in PHP pattern lab to introduce some of the same ideas? They specifically reference this project.

SiliconValet commented 7 years ago

yeah, Derek Reese is driving a lot of that and some of those ideas are coming from this effort.

On Fri, Sep 1, 2017 at 2:27 AM, Micah Godbolt notifications@github.com wrote:

I'm glad someone is enjoying and appreciating it! Have you heard about the movement in PHP pattern lab to introduce some of the same ideas? They specifically reference this project.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/PatternBuilder/pattern-kit/pull/20#issuecomment-326500849, or mute the thread https://github.com/notifications/unsubscribe-auth/AArcNqv_MW-RolAbE1lm7j9Usc5g5LHIks5sd6PMgaJpZM4PHrIp .

castastrophe commented 6 years ago

@SiliconValet & @micahgodbolt It seems like this MR is getting a little stale. I'm wondering what state of review this MR is in? Is it close to being mergeable? Does it change any functionality and if so, should we tie it to a major release?

micahgodbolt commented 6 years ago

I'm not really in a position to review it at the moment.

castastrophe commented 6 years ago

@micahgodbolt Understandable! I just saw you were in the discussion so I wanted to cc you. I was more asking @SiliconValet to see where this was in review.

micahgodbolt commented 6 years ago

Thanks :)

On Fri, Apr 6, 2018, 7:38 AM Cassondra notifications@github.com wrote:

@micahgodbolt https://github.com/micahgodbolt Understandable! I just saw you were in the discussion so I wanted to cc you. I was more asking @SiliconValet https://github.com/SiliconValet to see where this was in review.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PatternBuilder/pattern-kit/pull/20#issuecomment-379273443, or mute the thread https://github.com/notifications/unsubscribe-auth/ABXlTFhJQ8ikAZhrXar3Id-ZXzx7mgV-ks5tl33LgaJpZM4PHrIp .

starryeyez024 commented 6 years ago

@SiliconValet can you provide testing & setup instructions?

ThatGuyCND commented 5 years ago

@SiliconValet used composer-link to test this locally with WebRH's PK implementation. Everything works as expected. I would say that this PR is ready to merge.