Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Task 1,2,3 and partially 4 #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Task 1,2,3 and partially 4 #1

wants to merge 1 commit into from

Conversation

theRealNonso
Copy link

What:
I completed task 1,2,3, and partially 4.

How: I created a urls.py file in my article app, also updated the settings file to include the rest framework settings, and i also hid the secret key from public view using enviroment variables. To solve question 4, i added a models.PROTECT option to ensure when a tag is being deleted it is checked if an article is attached to it. If time were on my hand, I would using signals to implent the tag update check using the post_save method in signals. I also changed the generic views to Viewset to allow me flexibility. I could not complete my unit test because i clocked 2 hours

What: I completed task 1,2,3, and partially 4. How: I created a urls.py file in my article app, also updated the settings file to include the rest framework settings, and i also hid the secret key from public view using enviroment variables. To solve question 4, i added a models.PROTECT option to ensure when a tag is being deleted it is checked if an article is attached to it. If time were on my hand, I would using signals to implent the tag update check using the post_save method in signals. I also changed the generic views to Viewset to allow me flexibility. I could not complete my unit test because i clocked 2 hours
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant