Implementation related permissions and self-selection, planning and design #65 #68
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Creation of new command show_permissions Update of new checkboxes to use relatedperms
MichaelSuarez22
force-pushed
the
master
branch
from
8c33162 to
e867644
Compare
Delete comment in gtpermissions Delete comment in apps.py
new chanhes and new test in API_test
Delete unnecesary comments in permissionsmanagement
allexiusw
reviewed
Sep 22, 2023
Comment on lines
+31
to
+33
| #this unit test verifies the error handling behavior of an API view in Django by making a request | ||
| # to a dynamic URL with a specific resource_id value and ensuring that the view responds with a 404 status code | ||
| # if the resource is not found. |
There was a problem hiding this comment.
Suggested change
| #this unit test verifies the error handling behavior of an API view in Django by making a request | |
| # to a dynamic URL with a specific resource_id value and ensuring that the view responds with a 404 status code | |
| # if the resource is not found. |
djgentelella/tests/API_test.py
Outdated
Comment on lines
35
to
36
| class ErrorHandlingAPITestCase(TestCase): | ||
| def setUp(self): |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
djgentelella/tests/API_test.py
Outdated
Comment on lines
70
to
73
| #this unit test verifies that the update_related_permission function can correctly update | ||
| # existing related permissions and properly handles the situation | ||
| # where the related permission is not found in the list of existing permissions. | ||
|
|
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
djgentelella/tests/API_test.py
Outdated
| # Otras propiedades necesarias | ||
| ) | ||
|
|
||
| def test_retrieve_permission_related_detail(self): |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
djgentelella/tests/API_test.py
Outdated
Comment on lines
896
to
908
| class PermissionRelatedAPITest(TestCase): | ||
| def setUp(self): | ||
| self.client = APIClient() | ||
|
|
||
| def test_retrieve_nonexistent_permission_related_detail(self): | ||
| # Define un 'permission_id' que no existe en la base de datos | ||
| nonexistent_permission_id = 9999 | ||
|
|
||
| url = reverse('api_related_permissions_detail', kwargs={'permission_id': nonexistent_permission_id}) | ||
| response = self.client.get(url) | ||
|
|
||
| # Verifica que la respuesta tenga un código de estado 404 (Not Found) | ||
| self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND) |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Comment on lines
25
to
26
| <!-- Botones de Vista, Edición y Eliminación (si el usuario tiene los permisos correspondientes) --> | ||
|
|
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
djgentelella/tests/API_test.py
Outdated
Comment on lines
23
to
25
| ======= | ||
| from djgentelella.models import PermissionRelated | ||
| >>>>>>> nueva-rama |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
djgentelella/tests/API_test.py
Outdated
| import unittest | ||
| from django.test import TestCase | ||
| from django.conf import settings | ||
| <<<<<<< HEAD |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
djgentelella/tests/API_test.py
Outdated
| from django.test import TestCase | ||
| from django.conf import settings | ||
| <<<<<<< HEAD | ||
| from djgentelella.models import PermissionRelated # Reemplaza 'yourapp' con el nombre real de tu aplicación |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
djgentelella/tests/__init__.py
Outdated
| @@ -3,3 +3,4 @@ | |||
| from .StoryMap_Test import * | |||
| from .TimeLine_Test import * | |||
| from .Notification_Test import * | |||
| from .API_test import* | |||
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In this case a reimplementation and editing of the permissions system that contains the Django Gentelella widgets program was performed. Among its main modifications is for example the ability to create, delete, add and edit permissions along with other related permissions, it should be noted that each permission can have multiple permissions, a single permission or may not even have any associated permissions. In addition it is important to emphasize that to achieve the above is made use of the Django API rest framework, so you can bring the data of the permissions that are within the administration of Django. Likewise at the graphical interface level, just select a user and then select the permission you want to add, once you choose and check the permission, other checkboxes related to that permission are displayed or a message is displayed indicating that there are no related permissions. Also within the modal in which these permissions are present, there is a third section that will be used as a means of help and information for the user who manipulates the permissions system of the program.
For more information you can consult: #65