-
Notifications
You must be signed in to change notification settings - Fork 110
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
Mixed suggestions #400
Comments
@sandstrom great suggestion on the repo changes. Done.
Thank you. Feature suggestions: I'd suggest a new issue with title |
@jmeridth For discussions, I'd basically open the tab, and never look at it. Don't feel any obligation to answer anything or moderate -- I'd even say it's a mistake as a maintainer to spend any time on the discussions, better spend it on higher-leverage things. Basically it'll just be a place for the community to ask questions and answer others. Sort of like a 'stack overflow' scoped to this project. I've opened new issues, with some suggestions. Happy to discuss all of them in more detail!
Closing this issue now. |
@sandstrom that's fair re: discussions. I do think maintainers have to pay attention to it to ensure no one is violating the code of conduct. Hopefully things will get reported but not always. Thank you for splitting out the issues. It is appreciated. |
@jmeridth Just curious, have you had time to look at any of the issues listed above? Happy to hear your thoughts, and if any of them seems reasonable to you, me or a colleague can flesh out some more details (will take a few days), ping you on the issue again for your feedback/approval, and then put together a PR with tests. |
@sandstrom I'd suggest pinging @lessthanjacob @njbbaer @ritikesh on these. |
Thanks, will do! |
Is there an existing issue for this?
Is your feature request related to a problem? Please describe
Thanks for an awesome gem!
I've played around a bit with it recently, and it's very useful! 🎉
First some suggestions for the repo:
Some feature suggestions:
These are just sentence-level suggestions. If you are open to any of them, I can re-write these into something more concrete, e.g. with suggestions on method signature changes, or additional methods, to support these things.
AssociationExtractor
extractor, we could easily build it ourselves (see below).AssociationExtractor
with a custom extractor. Maybe one could allow extensions to replace both the association and field extractors?Blueprinter.configure
is global, if we optionally set this on a base class instead, it would make it easier to use multiple realms of blueprints, within a single code base.BlueprinterAbstractA < Blueprinter::Base
and thenBlueprinterAbstractB < Blueprinter::Base
, then something likeApi1UserBlueprinter < BlueprinterAbstractA
and for the other APIApi2UserBlueprinter < BlueprinterAbstractB
.if
Again, these are just jotted down. If you are open to considering them, I can provide more detailed suggestions.
The text was updated successfully, but these errors were encountered: