Skip to content

Conversation

@zfi
Copy link
Contributor

@zfi zfi commented May 16, 2018

When processing a request for a project record, the app was pulling the project's ancestor project, if one was defined, and performing all of the processing we do to a project record just to obtain a few static fields. It appears that these ancestor or parent fields are not used in the UI and are probably no longer required.

This patch refactors code to add method overrides to support an additional Boolean parameter to enable/disable the ancestor project record data in the JSON payload when retrieving a project record.

@zfi zfi requested a review from MatzElectronics May 16, 2018 21:43
Copy link
Collaborator

@MatzElectronics MatzElectronics left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay - my Java isn't too strong, but as far as I can tell it looks good.

@MatzElectronics MatzElectronics merged commit 9b34c7c into parallaxinc:0.100 May 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants