Limit draft phase update events to topcoder project members

[gondzo]

@vikasrohit this one fixes topcoder-platform/tc-notifications#85
we'll limit the notifications only to topcoder team if phase is not visible to customer (draft)

[vikasrohit]

Thank @gondzo It seems like we are still generating the events but just trying to hide them on front end by looking at allowedUser field?

[gondzo]

generating events - yes
generating notifications - only for users in allowedUsers list (no need to filter in connect-app or emails)

[vikasrohit]

generating notifications - only for users in allowedUsers list

Did I miss the changes in tc-notifications then?

[gondzo]

ugh, yes, I pushed straight to dev

topcoder-platform/tc-notifications@1764005
topcoder-platform/tc-notifications@1e80092

[vikasrohit]

Ohk. Got it.

[vikasrohit]

@gondzo I think there is some potential issue here. topcoder-platform/tc-notifications@1e80092#diff-c70adcc7eeb9e6a77abb22af0d363d5cR163 specifies allowedUser as string while topcoder-platform/tc-notifications@1e80092#diff-c70adcc7eeb9e6a77abb22af0d363d5cR203 as variable

[vikasrohit]

@gondzo I have patched (topcoder-platform/tc-notifications@d1eefb9) the tc-notifications to use string instead of variable name for the issue mentioned above, let me know if you think that is not a desired change.