Nick Sutterer comments

Results 675 comments of


                                            Nick Sutterer

XML changes: Avoid to set tag name from representer. Use as option instead.

Can you make an example how this helps, I didn't really understand it by looking at the tests. Thanks, man!

XML changes: Avoid to set tag name from representer. Use as option instead.

Sorry to disappoint you, Alex, but XML support in Representable has a really low priority right now in my life - I promise you to look over it this week!...

XML changes: Avoid to set tag name from representer. Use as option instead.

This PR adds the `:as` option so the wrap for nested representers can be specified from the composing representer. It doesn't look at the `representation_wrap` if `:as` is provided. Is...

XML changes: Avoid to set tag name from representer. Use as option instead.

Yeah, I remember `:as` was there, before. But this didn't work with nested representers??

XML changes: Avoid to set tag name from representer. Use as option instead.

I sloooooowly start to remember this problem! It's great you came up with it! :+1:

XML changes: Avoid to set tag name from representer. Use as option instead.

I am working on fixing this - it's a terrible mess in the current implementation and I might have to break the API. So, to sum up what you need....

XML changes: Avoid to set tag name from representer. Use as option instead.

I think there's two problems in the current XML implementation. 1. As you said, the `as:` option doesn't work - I think this would fix your problem. 2. We support...

Generator does not match trailblazer concept conventions

Thanks Paul, I believe the generator now sits in https://github.com/trailblazer/trailblazer-generator !

Generator does not match trailblazer concept conventions

Ultimately, we want to drop Cells `ViewModel` and `Concept` and only support `Trailblazer::Cell` - which means that... we somehow would've to merge/reference... uhm....

Using the :skip_parse option with JSON::Hash values produces Pipeline::Stop

Hi @jpsikorra I kind of gave up on Representable, we will soon publish `transform` which will suit all your needs (FYI).