Merge branch 'feature/STACKN-51' into serve-path-based

Author: stefanhellander

components/studio/models/templates/models_details.html

@@ -72,6 +72,40 @@ <h2>{{ model.name }}</h2>
                     {% endif %}
                 </div>
             </div>
+            <form method="POST" action="{% url 'models:change_access' request.user model.project.slug model.id %}">
+                {% csrf_token %}
+
+                <div class="d-flex bd-highlight mb-3 col-md-6">
+                    <label for="id_access" class="p-2 bd-highlight" style="font-weight: bold;">
+                        Visibility
+                    </label>
+                    <select id="id_access" type="text" name="access" maxlength="2" required
+                            class="ml-auto form-control form-control-sm" style="width: 20%;">
+                        <option selected>
+                            {% if model.access == 'PR' %}
+                            Private
+                            {% elif model.access == 'PU' %}
+                            Public
+                            {% else %}
+                            Limited
+                            {% endif %}
+                        </option>
+                        {% for choice in model_access_choices %}
+                        {% if choice == 'PR' %}
+                        <option value="PR">Private</option>
+                        {% elif choice == 'PU' %}
+                        <option value="PU">Public</option>
+                        {% else %}
+                        <option value="LI">Limited</option>
+                        {% endif %}
+                        {% endfor%}
+                    </select>
+                </div>
+                <div class="d-flex bd-highlight mb-3 col-md-6"
+                     style="border-top: 1px solid #dee2e6; padding-top: 20px;">
+                    <button type="submit" class="ml-auto btn btn-primary">Save</button>
+                </div>
+            </form>
         </div>
         <div class="tab-pane fade" id="deployments" role="tabpanel" aria-labelledby="profile-tab">
             {% if deployments %}

components/studio/models/urls.py

@@ -11,4 +11,5 @@
     path('<user>/<project>/models/<int:id>', views.details, name='details'),
     path('models/<int:id>', views.details_public, name='details_public'),
     path('<user>/<project>/models/<int:id>/delete', views.delete, name='delete'),
+    path('<user>/<project>/models/<int:id>/access', views.change_access, name='change_access'),
 ]

components/studio/models/views.py

@@ -60,10 +60,26 @@ def create(request, user, project):
         return render(request, template, locals())
 
 
+@login_required(login_url='/accounts/login')
+def change_access(request, user, project, id):
+    model = Model.objects.filter(pk=id).first()
+
+    if request.method == 'POST':
+        visibility = request.POST.get('access', '')
+        if visibility != model.access:
+            model.access = visibility
+            model.save()
+
+    return HttpResponseRedirect(
+        reverse('models:details', kwargs={'user': user, 'project': project, 'id': id}))
+
+
 @login_required(login_url='/accounts/login')
 def details(request, user, project, id):
     project = Project.objects.filter(slug=project).first()
     model = Model.objects.filter(id=id).first()
+    model_access_choices = ['PU', 'PR', 'LI']
+    model_access_choices.remove(model.access)
     deployments = DeploymentInstance.objects.filter(model=model)
 
     report_generators = ReportGenerator.objects.filter(project=project)