blueprinter icon indicating copy to clipboard operation
blueprinter copied to clipboard

Exclude a view functionality

Open Abhinaya-P opened this issue 6 years ago • 2 comments

Though excludes can be used to exclude several fields. If all the excluded fields are from a single view, then exclude_view is a more refined and usable option

@mcclayton , @philipqnguyen

Abhinaya-P avatar May 02 '19 15:05 Abhinaya-P

Thank you so much for contributing @Abhinaya-P, I'll try to get some time soon to take a look at this, but I'll be on vacation soon. May have to check it out when I get back next week.

mcclayton avatar May 06 '19 23:05 mcclayton

Hi @Abhinaya-P, Thank you for both your contribution to Blueprinter and your patience with us as we look into this. To provide some insight, there are a couple things holding us up on this PR currently.

  1. This PR has exposed some potential undefined behavior and edge cases in our current exclude field functionality which we are looking into and trying to address before we can determine exactly how exclude_view should behave in the similar edge cases. i.e. we have not clearly defined the order of operations of excluding/including fields -- and so when we have multiple views including/excluding fields, we are still determining what the order of operations/behavior should be.
  2. We are still considering if exclude_view is functionality we want to introduce into the Blueprinter gem.

mcclayton avatar Jul 24 '19 23:07 mcclayton

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Nov 19 '23 01:11 github-actions[bot]