Skip to content
This repository has been archived by the owner on Jul 18, 2019. It is now read-only.

Academic information module so far #84

Open
wants to merge 66 commits into
base: master
Choose a base branch
from

Conversation

narolkr
Copy link
Contributor

@narolkr narolkr commented Nov 7, 2017

No description provided.

course = ""
exam_t = ""
timetable = ""
pass
Copy link
Contributor

Choose a reason for hiding this comment

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

This view function will not move ahead due to this pass

Copy link
Contributor Author

Choose a reason for hiding this comment

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

is there a way to use ajax while pressing the side tabs? Because when i use the tabs that are in {% include xyz.html %} file, it opens for the first time but after that it doesn't.

Copy link
Contributor

Choose a reason for hiding this comment

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

I talked to @anuraagheisenberg and he said, he helped resolve it. I hope it's done.

s_id = request.POST.get('student_id')
c_id = request.POST.get('course_id')
print(s_id)
print(c_id)
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove/comment all the debugging print


def get_attendance(request):
course_id = request.GET.get('course_id')
print(course_id)
Copy link
Contributor

Choose a reason for hiding this comment

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

same

return HttpResponse("Data Inputed")


def delete1(request):
Copy link
Contributor

Choose a reason for hiding this comment

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

Give descriptive names and please add documentation comments.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants