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

feat: implement stage1 API #8

Merged
1 commit merged into from
Nov 26, 2024
Merged

feat: implement stage1 API #8

1 commit merged into from
Nov 26, 2024

Conversation

TakalaWang
Copy link
Contributor

@TakalaWang TakalaWang commented Nov 26, 2024

This pull request includes significant changes to the src/lib and src/routes directories to update the data structures and enhance functionality for handling sessions and discussions. The most important changes include modifying the data structures for sessions, individual discussions, and group discussions, as well as adding new functions and endpoints to handle these changes.

Changes to data structures:

  • src/lib/types/IndividualDiscussion.ts: Added goal, subQuestions, and resourcesTexts fields to FirestoreIndividualDiscussion and IndividualDiscussion interfaces. Updated convertFirestoreIndividualDiscussion to include these new fields. [1] [2] [3]
  • src/lib/types/groupDiscussion.ts: Added groupName field to FirestoreGroupDiscussion and GroupDiscussion interfaces. Updated convertFirestoreGroupDiscussion to include this new field. [1] [2] [3]
  • src/lib/types/session.ts: Added stage, resourceIds, and groups fields to FirestoreSession and Session interfaces. Updated convertFirestoreSession to include these new fields. [1] [2] [3]

Enhancements to functionality:

Refactoring and cleanup:

New utility function:

@JacobLinCool JacobLinCool closed this pull request by merging all changes into main in 7206796 Nov 26, 2024
@JacobLinCool JacobLinCool deleted the stage1-api branch November 26, 2024 22:24
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