Skip to content

Commit dd08081

Browse files
author
Vikas Agarwal
committed
Github issue #572, Messaging: Owner response post missing from status update
-- Removed the constraint of user to be a team member of the project in order to view the comments of status updates. Further, I am assuming we need to do the same for Discussions tab as well, so removing the constraint there as well. So, now any user who has the access to the project's dashboard and discussions pages, can view posts and comments.
1 parent cb4094e commit dd08081

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/projects/detail/containers/FeedContainer.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ class FeedView extends React.Component {
212212
<div className="feed-action-card" key={item.id}>
213213
<Feed
214214
{...item}
215-
allowComments={ item.allowComments && !!currentMemberRole}
215+
allowComments={ item.allowComments }
216216
currentUser={currentUser}
217217
onNewCommentChange={this.onNewCommentChange.bind(this, item.id)}
218218
onAddNewComment={this.onAddNewComment.bind(this, item.id)}

src/projects/detail/containers/MessagesContainer.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,7 @@ class MessagesView extends React.Component {
299299
return (
300300
<MessageDetails
301301
{...activeThread}
302-
allowAddingComment={ activeThread.allowComments && !!currentMemberRole }
302+
allowAddingComment={ activeThread.allowComments }
303303
onLoadMoreMessages={this.onShowAllComments.bind(this, activeThread.id)}
304304
onNewMessageChange={this.onNewMessageChange}
305305
onAddNewMessage={ this.onAddNewMessage.bind(this, activeThread.id) }

0 commit comments

Comments
 (0)