[PPS] Added endpoint to get all reports of student in JSON #20

pritamps · 2023-03-09T12:22:08Z

Added a new secondary index to dynamodb -- our reports db is now queryable by userid as well
Added a new endpoint: /reports/student_reports/4444444444?format=json
Sample output:

{"student_id":"4444444444","reports":[{"test_name":"Test - Homework Quiz","test_session_id":"Test_-_Homework_Quiz_2022-07-27_62e246c6ed7e4cbb6a78b823","rank":"1.0","percentile":80,"report_link":"https://reports.avantifellows.org/reports/student_quiz_report/Test_-_Homework_Quiz_2022-07-27_62e246c6ed7e4cbb6a78b823/4444444444"},{"test_name":"Test - Homework Quiz","test_session_id":"Test_-_Homework_Quiz_2022-07-27_62e246c6ed7e4cbb6a78b823","rank":"1.0","percentile":90,"report_link":"https://reports.avantifellows.org/reports/student_quiz_report/Test_-_Homework_Quiz_2022-07-27_62e246c6ed7e4cbb6a78b823/4444444444"},{"test_name":"Test - Homework Quiz","test_session_id":"Test_-_Homework_Quiz_2022-07-27_62e246c6ed7e4cbb6a78b823","rank":"2.0","percentile":80,"report_link":"https://reports.avantifellows.org/reports/student_quiz_report/Test_-_Homework_Quiz_2022-07-27_62e246c6ed7e4cbb6a78b823/4444444444"},{"test_name":"Test - Homework Quiz","test_session_id":"Test_-_Homework_Quiz_2022-07-27_62e246c6ed7e4cbb6a78b823","rank":"3.0","percentile":40,"report_link":"https://reports.avantifellows.org/reports/student_quiz_report/Test_-_Homework_Quiz_2022-07-27_62e246c6ed7e4cbb6a78b823/4444444444"}]}
``

generate_table/__main__.py

suryabulusu · 2023-03-16T07:06:12Z

generate_table/__main__.py

+
+
+def drop_secondary_ind():
+    drop_secondary_index("gsi_user_id-section")


shouldn't this be "gsi_user_Id"

So these methods are meant to be run manually, mostly one-time. Will actually make this a parameter for the function.

suryabulusu · 2023-03-16T07:10:36Z

generate_table/__main__.py

@@ -35,5 +40,15 @@ def drop_tables():
    drop_student_quiz_reports(ddb)


+def add_secondary_ind():
+    # DON'T RUN THIS for the key user_id key as it's already been run


but this is getting run by default in `if name == "main" ? 🤔

Added comment explaining things

suryabulusu · 2023-03-21T08:18:32Z

generate_table/__main__.py

+    """
+    Drops a secondary index
+    """
+    drop_secondary_index(index_name)


drop_secondary_index(ddb, index_name)

and ddb = initialize_db()

[PPS] Added endpoint to get all reports of student in JSON

2b84c9d

pritamps had a problem deploying to Staging March 9, 2023 12:22 — with GitHub Actions Failure

pritamps requested review from deepansh96, sudheshna-donthineni and suryabulusu March 9, 2023 12:24

suryabulusu reviewed Mar 16, 2023

View reviewed changes

generate_table/__main__.py Outdated Show resolved Hide resolved

suryabulusu reviewed Mar 16, 2023

View reviewed changes

[PPS] Changes according to PR

88256f0

pritamps had a problem deploying to Staging March 21, 2023 08:14 — with GitHub Actions Failure

suryabulusu reviewed Mar 21, 2023

View reviewed changes

[PPS] Oops

bfc8fdc

pritamps temporarily deployed to Staging March 21, 2023 08:23 — with GitHub Actions Inactive

suryabulusu approved these changes Mar 21, 2023

View reviewed changes

pritamps merged commit e9a49a3 into main Mar 21, 2023

pritamps deleted the add-endpoint-for-all-student-reports branch March 21, 2023 08:28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[PPS] Added endpoint to get all reports of student in JSON #20

[PPS] Added endpoint to get all reports of student in JSON #20

pritamps commented Mar 9, 2023 •

edited

Loading

suryabulusu Mar 16, 2023

pritamps Mar 16, 2023

suryabulusu Mar 16, 2023

pritamps Mar 21, 2023

suryabulusu Mar 21, 2023

suryabulusu Mar 21, 2023



		def drop_secondary_ind():
		drop_secondary_index("gsi_user_id-section")

[PPS] Added endpoint to get all reports of student in JSON #20

[PPS] Added endpoint to get all reports of student in JSON #20

Conversation

pritamps commented Mar 9, 2023 • edited Loading

suryabulusu Mar 16, 2023

Choose a reason for hiding this comment

pritamps Mar 16, 2023

Choose a reason for hiding this comment

suryabulusu Mar 16, 2023

Choose a reason for hiding this comment

pritamps Mar 21, 2023

Choose a reason for hiding this comment

suryabulusu Mar 21, 2023

Choose a reason for hiding this comment

suryabulusu Mar 21, 2023

Choose a reason for hiding this comment

pritamps commented Mar 9, 2023 •

edited

Loading