Optimisation - project detail api call can be optimized by avoiding attachment detail.

[sachin-maheshwari]

Current call is fetching 'attachment' details along with project detail. For 'attachment' detail, there is another back-end API (ap-file service) call, which can be avoided as we are not using 'attachment' detail anywhere in notification, we are using only associated members with project to send notifications.
https://github.com/topcoder-platform/tc-notifications/blob/dev/connect/service.js#L21

[sachin-maheshwari]

@vikasrohit can you please verify whatever I mentioned in description is doable ?

[vikasrohit]

@sachin-maheshwari I think we need the attachment details now as we are sending attachment's Url in bus events. Further, to call ap-file-service, we still need attachment id which is retrieved only by fetching the attachment details.
And lastly we support avoiding an extra db call to attachments by not passing attachments in the fields param while retrieving project details. We do not fetch attachments when listing the projects but do fetch it when showing project details. :)
However, thanks for the suggestion. I found one issue though.