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

new params to quizzes #14

Closed
MasterPi-2124 opened this issue Jun 6, 2023 · 4 comments · Fixed by #16
Closed

new params to quizzes #14

MasterPi-2124 opened this issue Jun 6, 2023 · 4 comments · Fixed by #16
Assignees
Labels
bug Something isn't working enhancement Improve existing feature, function feature Feature request

Comments

@MasterPi-2124
Copy link
Owner

each quiz should also have these params:

  • status: Incoming, happening, and finished. the status can be achieved by comparing startTime, endTime and current time
  • QR raw image URL. QR logic should be executed right from back end
    proposed changes:
  • change classID type from string to int
  • if possible, change ID to normal number, starting from 1, not random string
  • __v is unused
@MasterPi-2124 MasterPi-2124 added bug Something isn't working enhancement Improve existing feature, function feature Feature request labels Jun 6, 2023
@nghiatm341
Copy link
Collaborator

nghiatm341 commented Jun 6, 2023

each quiz should also have these params:

  • status: Incoming, happening, and finished. the status can be achieved by comparing startTime, endTime and current time
  • QR raw image URL. QR logic should be executed right from back end
    proposed changes:
  • change classID type from string to int
  • if possible, change ID to normal number, starting from 1, not random string
  • __v is unused

@MasterPi-2124

  • classID để dành cho lưu cái id được tạo ra để liên kết, tìm kiếm, không phải mã lớp, cái name mới chứa mã lớp (hoặc mình có thể thệm trường riêng cho nó)
  • Status, ok để mình xem
  • __v là mặc định có, mongoose khuyên là không nên bỏ, trừ khi mình biết mình đang làm gì (mình có thể ẩn nó đi để front không nhận nó nữa)
  • Mình chưa hiểu ý bạn về phần QR, bạn giải thích rõ hơn được không?

@MasterPi-2124
Copy link
Owner Author

  • ô cứ tưởng classId là mã lớp :))) Nếu vậy mình nghĩ nên đổi lại tên param cho đỡ nhầm lẫn, classID sẽ là mã lớp của quiz, còn ID có thẻ là id bạn đang define cũng đc
  • sau khi 1 quiz đc tạo trên backend, 1 link đến quiz sẽ đc tự động gen ra để sv truy cập vào, và 1 mã QR của URL sẽ đc gen ra và để trong api.

@nghiatm341
Copy link
Collaborator

  • ô cứ tưởng classId là mã lớp :))) Nếu vậy mình nghĩ nên đổi lại tên param cho đỡ nhầm lẫn, classID sẽ là mã lớp của quiz, còn ID có thẻ là id bạn đang define cũng đc
  • OK, thực ra quiz là cái mình tạo ra đầu tiên, nó còn nhiều vấn đề nên mình định tranh thủ lần này để sửa lại nhiều thứ
  • sau khi 1 quiz đc tạo trên backend, 1 link đến quiz sẽ đc tự động gen ra để sv truy cập vào, và 1 mã QR của URL sẽ đc gen ra và để trong api.
  • Thực ra các link / QR để sinh viên truy cập vào nên dẫn tới một trang của front để sinh viên điền thông tin điểm danh vào, khi sinh viên nhấn OK thì front sẽ gọi api tạo quizRecord để lưu thông tin sinh viên điền (lưu lượt điểm danh) rồi chuyển tiếp sinh viên đến form trắc nghiệm (nếu có) hoặc chỉ hiển thị thông báo thành công và "Bạn có thể đóng tab này" chẳng hạn

@nghiatm341
Copy link
Collaborator

Lấy danh sách quiz có thêm bộ lọc theo class

nghiatm341 added a commit that referenced this issue Jun 7, 2023
- finishing issue #14
@nghiatm341 nghiatm341 linked a pull request Jun 7, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement Improve existing feature, function feature Feature request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants