From 231521f53c42c5e096bccef2562127e83ff5d1bb Mon Sep 17 00:00:00 2001 From: Kevin Mehall Date: Mon, 25 Nov 2013 17:47:41 -0500 Subject: [PATCH] Link to the stream only in the "subscribed" PM, not on the announce stream. Until we can add a banner to help users subscribe, it may be confusing to narrow to a stream where you are not subscribed. Partial revert of 390bdef (imported from commit ea75fc59b979589b975465a3fecffea0f014fcf6) --- zerver/views/__init__.py | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/zerver/views/__init__.py b/zerver/views/__init__.py index 74709ca619..52b8b26b49 100644 --- a/zerver/views/__init__.py +++ b/zerver/views/__init__.py @@ -1701,15 +1701,9 @@ def add_subscriptions_backend(request, user_profile, if notifications_stream is not None: if len(created_streams) > 1: stream_msg = "the following streams: %s" % \ - (", ".join('[%s](%s)' % ( - s.name, - stream_link(s.name), - ) for s in created_streams),) + (", ".join('`%s`' % (s.name,) for s in created_streams),) else: - stream_msg = "a new stream [%s](%s)" % ( - created_streams[0].name, - stream_link(created_streams[0].name) - ) + stream_msg = "a new stream `%s`" % (created_streams[0].name) msg = ("%s just created %s. To join, visit your [Streams page](#subscriptions)." % (user_profile.full_name, stream_msg)) notifications.append(internal_prep_message(settings.NOTIFICATION_BOT,