https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292021-04-06T16:05:57ZRedmineRedmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1018662021-04-06T16:05:57ZGo MAEDA
<ul></ul><p>The following code is a sample implementation.</p>
<pre><code class="diff syntaxhl"><span class="gh">diff --git a/app/views/projects/_members_box.html.erb b/app/views/projects/_members_box.html.erb
index e915ab910..72d7fb97b 100644
</span><span class="gd">--- a/app/views/projects/_members_box.html.erb
</span><span class="gi">+++ b/app/views/projects/_members_box.html.erb
</span><span class="p">@@ -4,5 +4,10 @@</span>
<% @principals_by_role.keys.sort.each do |role| %>
<p><span class="label"><%= role %>:</span> <%= @principals_by_role[role].sort.collect{|p| link_to_user p}.join(", ").html_safe %></p>
<% end %>
<span class="gi">+ <% if @project.is_public %>
+ <hr>
+ <p span class="label"><%= l(:field_is_public) %>:</span> <%= l(:general_text_Yes) %>
+ <em class="info"><%= Setting.login_required? ? l(:text_project_is_public_non_member) : l(:text_project_is_public_anonymous) %></em>
+ <% end -%>
</span> </div>
<% end %>
</code></diff></pre> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1018672021-04-06T16:07:09ZGo MAEDA
<ul><li><strong>Related to</strong> <i><a class="issue tracker-3 status-5 priority-4 priority-default closed" href="/issues/27009">Patch #27009</a>: Clarify consequences of disabling the login_required setting</i> added</li></ul> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1018702021-04-06T23:57:39ZMizuki ISHIKAWA
<ul></ul><p>+1</p>
<p>Private information may be leaked if the settings are mistakenly made public when creating a project.<br />I think this feature is needed to quickly notice that the project is open to the public.</p> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1018712021-04-07T00:46:01ZKevin Fischer
<ul></ul><p>+1</p>
<p>On a somewhat related note:<br />About 1 month ago we also implemented and posted a patch for a new permission for publishing projects in response to <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Disable public project creation on a role basis (Closed)" href="https://www.redmine.org/issues/9029">#9029</a> . Maybe this could be considered for Redmine 5.0 ? It would surely help to counter the security concerns regarding public projects....</p> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1023302021-05-03T21:13:06ZMarius BĂLTEANU
<ul></ul><p>Maybe we should add a Public badge next to Overview title as we have for issues and versions.</p> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1023662021-05-08T07:27:51ZMizuki ISHIKAWA
<ul><li><strong>File</strong> <a href="/attachments/27326">0001-Add-test.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/27326/0001-Add-test.patch">0001-Add-test.patch</a> added</li><li><strong>File</strong> <a href="/attachments/27327">0002-Add-project-public-badge.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/27327/0002-Add-project-public-badge.patch">0002-Add-project-public-badge.patch</a> added</li><li><strong>File</strong> <a href="/attachments/27329">screenshot.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/27329/screenshot.png">screenshot.png</a> added</li></ul><p>I have attached a patch that adds a test to <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: Show notice on project's overview page when the project is public (New)" href="https://www.redmine.org/issues/35044#note-1">#35044#note-1</a>. => <a class="attachment" href="https://www.redmine.org/attachments/27326">0001-Add-test.patch</a></p>
<p>I also made <a class="attachment" href="https://www.redmine.org/attachments/27327">0002-Add-project-public-badge.patch</a> to add a badge based on the opinion of <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: Show notice on project's overview page when the project is public (New)" href="https://www.redmine.org/issues/35044#note-5">#35044#note-5</a>. Please let me know if you need to separate the issue.<br /><img src="https://www.redmine.org/attachments/download/27329/screenshot.png" style="width: 500px;border: 1px solid #ccc;" alt="" /></p>
<p>These patches were developed with <a href="https://redmine.org/users/388654" class="external">Ko Nagase</a> and <a href="https://www.redmine.org/users/86605" class="external">Akihiro MATOBA</a> .</p> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1023682021-05-08T08:20:37ZGo MAEDA
<ul></ul><p>Mizuki ISHIKAWA wrote:</p>
<blockquote>
<p>I have attached a patch that adds a test to <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: Show notice on project's overview page when the project is public (New)" href="https://www.redmine.org/issues/35044#note-1">#35044#note-1</a>. => <a class="attachment" href="https://www.redmine.org/attachments/27326">0001-Add-test.patch</a></p>
<p>I also made <a class="attachment" href="https://www.redmine.org/attachments/27327">0002-Add-project-public-badge.patch</a> to add a badge based on the opinion of <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: Show notice on project's overview page when the project is public (New)" href="https://www.redmine.org/issues/35044#note-5">#35044#note-5</a>. Please let me know if you need to separate the issue.<br /><img src="https://www.redmine.org/attachments/download/27329/screenshot.png" style="width: 500px;border: 1px solid #ccc;" alt="" /></p>
</blockquote>
<p>Thank you for the patch but I think it is better to post as a new issue because the feature implemented by the patch is very different from the original proposal.</p>
<p>In addition, I suggest the following improvements:</p>
<ul>
<li>The badge is for a single project, the noun should be singular ("projects" -> "project")</li>
<li>Maybe "public project" is too long. I think "public" is enough</li>
<li>The color #d22 is too prominent. Also, the red color gives a wrong impression that the public project itself is dangerous and bad. Making a project public is not dangerous. The danger is that you will set it up incorrectly</li>
</ul> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1023742021-05-10T00:41:39ZMizuki ISHIKAWA
<ul></ul><p>Go MAEDA wrote:</p>
<blockquote>
<p>Thank you for the patch but I think it is better to post as a new issue because the feature implemented by the patch is very different from the original proposal.</p>
<p>In addition, I suggest the following improvements:</p>
<ul>
<li>The badge is for a single project, the noun should be singular ("projects" -> "project")</li>
<li>Maybe "public project" is too long. I think "public" is enough</li>
<li>The color #d22 is too prominent. Also, the red color gives a wrong impression that the public project itself is dangerous and bad. Making a project public is not dangerous. The danger is that you will set it up incorrectly</li>
</ul>
</blockquote>
<p>Thank you for your feedback.<br />I made a new issue <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: Show public badge on project's overview page when the project is public (New)" href="https://www.redmine.org/issues/35221">#35221</a> and attached a new patch that was modified based on the feedback.</p> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1023862021-05-11T05:47:02ZMarius BĂLTEANU
<ul></ul><p>We should show this info only once on this page and the members box is not the best option because the project visibility is an attribute of the project, not of the members. Also, in projects with multiple members or news, the information will be visibile only after more or less scroll.</p>
<p>From my point of view, having the bagde with a title attribute should be enough. If you don’t like this approach, then we should show this info in the project attributes box (top left).</p> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1023872021-05-11T05:54:29ZMarius BĂLTEANU
<ul></ul><p>One more thing, project content visibility depends on the rights configured for anonymous/non member roles, maybe the message should be more specific because the current one says that the content is publicly available. Also, in an era of SaaS/cloud applications, we should avoid using the word network. .</p> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1023882021-05-11T06:19:10ZGo MAEDA
<ul></ul><p>What I wanted to show in the members box is public not whether the project is public or not, but anonymous users and nonmembers can access the project.</p>
<p>I think it would be good to discuss how to indicate that a project is public, based on the patch in <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: Show public badge on project's overview page when the project is public (New)" href="https://www.redmine.org/issues/35221">#35221</a>.</p> Redmine - Feature #35044: Show notice on project's overview page when the project is publichttps://www.redmine.org/issues/35044?journal_id=1023892021-05-11T06:26:29ZMarius BĂLTEANU
<ul></ul><p>Go MAEDA wrote:</p>
<blockquote>
<p>What I wanted to show in the members box is public not whether the project is public or not, but anonymous users and nonmembers can access the project.</p>
</blockquote>
<p>I understand now and I think it's a valuable information.</p>
<blockquote>
<p>I think it would be good to discuss how to indicate that a project is public, based on the patch in <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: Show public badge on project's overview page when the project is public (New)" href="https://www.redmine.org/issues/35221">#35221</a>.</p>
</blockquote>
<p>Ok, I will add my proposals there in the following days.</p>