-
Notifications
You must be signed in to change notification settings - Fork 19
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
progress pr - Batch requests in task table #10382
Conversation
Code Climate has analyzed commit 94b8d14f and detected 4 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
…fairs/caseflow into Batch-Requests-In-Task-Table
…artment-of-veterans-affairs/caseflow into Batch-Requests-In-Task-Table
render(error.serialize_response) | ||
end | ||
|
||
def build_document_counts_hash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@monfresh do you have any thoughts on how to get code climate to not complain about this having 7 statements?
render json: { most_recently_held_hearings_by_id: build_most_recently_held_hearings_hash } | ||
end | ||
|
||
def build_most_recently_held_hearings_hash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@monfresh same here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will take a look
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's going to be alot of changes to this, so please hold off for now i'll reach back out when its ready if i'm still facing this issue, thank you!
94b8d14
to
c39ab29
Compare
Resolves #8934