Skip to content

Create variousteams dashboard #74

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

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

Conversation

fe-cmd
Copy link

@fe-cmd fe-cmd commented Apr 1, 2025

  1. addition of "dashboard to installed apps"
  2. creation of urls to route functionalities to frontend views
  3. creation of models and views functions for various dashboards for staffs, volunteers and sponsorship management procedures
  4. regulation and seperation of various different teams authenticaations

these are sample setups but to further work on we can improve templates rendering and testing to ensure work flow.
@Mariatta @@cmaureir kindly please review this, thank you.

Copy link
Contributor

@Mr-Sunglasses Mr-Sunglasses left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Please remove __pycache__ folder.

To remove pycache file now after commit please reference to this stackoverflow link: https://stackoverflow.com/a/71930352/16563091

@@ -0,0 +1,26 @@
#model schema creation for more functionality addition to sponsorship portal
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see a new model here but no migrations for SponsorshipApplication model. Please Add it.

Steps to add it

make migrations

after that

make migrate

and commit the migrations file.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alright sir, i'll rectify this as well and get back to you

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hello @Mr-Sunglasses i have removed the pycache/ folder , and made the migrations and migrated for the sponsorship application models and also done the commit as well pls review thank you

@@ -0,0 +1,18 @@
from django.db import models
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see a new model here but no migrations file for SponsorshipApplication , VolunteerTask , StaffTask . Please Add it.

Steps to add it

make migrations

after that

make migrate

and commit the migrations file.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright sir , i was actually planning to do all that already as well, i just gave some samples code for you to see.
I'll correct that right away sir and get back to you

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hello @Mr-Sunglasses made the migrations and migrated for the 3 models and also done the commit as well pls review thank you very much

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Mr-Sunglasses i saw that you requested a change , please what do you mean, i dont really understand, do you want an access to the files uploaded here

@@ -11,12 +11,20 @@
"""

import os
import environ
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might have missed adding the dependency for django-environ, and it should be added to requirements.txt

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Content seems to be missing.

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.

3 participants