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

KrishnaManohar1997 - 💯 Task Completed #7

Open
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

KrishnaManohar1997
Copy link

What's Changed 💯

  • Created Base Model (Which consists commonly used fields) for Inheritance (Code Reusability)
  • Structured the project similar to MVC Architecture
    • Every Model is defined in a single File - To increase readability
    • Same goes with Serializers
  • Updated documentation for every feature updated with corresponding cURL's
  • Created a Common module, which include base models and constants utilized in the project.
  • Utilized base responses for reusability and faster development process
  • Followed PEP8 standards with flake8 linter and black formatter
  • Filtering, Ordering to existing Article API
  • Tests for the updated Article API's
  • Added Tag Creation/List/Updating/Deleting Functionality
  • Tag articles

Next Steps

  • Add Authentication
  • Change CORS and ALLOWED_HOSTS
  • Align project with Django's Checklist for Production Ready items

Manohar-CAW and others added 23 commits May 16, 2021 10:30
Add Schema for Tag Model with Create-List View
@KrishnaManohar1997 KrishnaManohar1997 changed the title KrishnaManohar1997 KrishnaManohar1997 - 💯 Task Completed May 16, 2021
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.

2 participants