Update Progress - Ziqing

students/ziqing26/knowledge.md

@@ -1,3 +1,14 @@
+## Data Migration
+
+* Before data migration we need to make sure we are clear on the ERD and relationship mapping.
+* To test scripts on data migration, we set up a staging Cloud SQL.
+* Currently there is no way to persist data bundle in datastore. We are still figuring out a proper way to test it.
+* Data validation after migration such as ensuring Foreign Keys are mapped correctly is important. We will explore more on that
+
+## Terraform
+
+* Saves a lot of time for deploying staging server. It will definitely save a lot of time if we have a shared staging server for testing. However TEAMMATEs might not be operating at a large enough scale to see the benefits.
+
 ## Docker
 A container is a standard unit of software that packages up code and all its dependencies so the application runs quickly and reliably from one computing environment to another. It is an isolated environment for code. This means that a container has no knowledge of the local operating system, or the local files.
 

students/ziqing26/observations.md

@@ -1,11 +1,28 @@
-### Project: Foo
+### Project: Refine
+
+[Refine](https://github.com/refinedev/refine) is a React Framework for building internal tools, admin panels, dashboards & B2B apps with unmatched flexibility. It uses TypeScript. It simplifies the development process and eliminate repetitive tasks by providing industry-standard solutions for crucial aspects of a project, including authentication, access control, routing, networking, state management, and i18n.
 
-Give an intro to the project here ...
 
 ### My Contributions
 
 Give a description of your contributions, including links to relevant PRs
 
+PR 1: [docs(core): add DataProvider interface definition #5653](https://github.com/refinedev/refine/pull/5653)
+
+Initially I thought contributing to documentation is easy, but I realize that contributing to documentation requires good understanding of the codebase structure and the workflow.
+
 ### My Learning Record
 
-Give tools/technologies you learned here. Include resources you used, and a brief summary of the resource.
+Give tools/technologies you learned here. Include resources you used, and a brief summary of the resource.
+
+[Refine Contributing Guide](https://refine.dev/docs/guides-concepts/contributing/)
+
+The PR review process in Refine is surprisingly fast. My PR is reviewed within one week.
+
+Observations of contributing process:
+
+Issues that are labelled good-first-issues often have comments that ask to be assigned the tasks. But the maintainers tend to take long to reply them. At the time they got back to potential contributers, contributers might already not be interested in it.
+
+Refine has a `Changeset` system where contributors need to label the impact on packages such as whether it requires a major version bump in any packages, as Refine uses a monorepo structure.
+
+Refine is well-documented and its core team is quite active in issues, which is a hugh advantage for first time contributers because their questions got answered immediately. However, their good-first-issues still have high barriers to entry because of the complicated code structure. That being said, their maintainers make a good effort to explain what might need to be done to submit a PR in the issues, making it easier to understand.

students/ziqing26/progress.md

@@ -8,3 +8,16 @@
 | 2 | Reviewed PR: [[#12048] Migrate SearchInstructorsAction #12340](https://github.com/TEAMMATES/teammates/pull/12340)|
 | 2 | Match CS3282 mentors with CS3281 students to facilitate their onboarding tasks.
 | 3 | Reviewed PR: [[#12048] Migrate GetOngoingSessionsAction for V9 #12710](https://github.com/TEAMMATES/teammates/pull/12710)|
+| 4 | Reviewed PR: [[#12048] Fix update and create feedbackquestion actions #12716](https://github.com/TEAMMATES/teammates/pull/12716) |
+| 5 | Reviewed PR: [[#12048] Migrate create account action #12702](https://github.com/TEAMMATES/teammates/pull/12702) |
+| 5 | Reviewed PR: [[#12048] Migrate instructor search indexing worker action #12731](https://github.com/TEAMMATES/teammates/pull/12731) |
+| 5 | Reviewed PR: [[#12048] Migrate PutDataBundleDocumentsAction #12734](https://github.com/TEAMMATES/teammates/pull/12734)|
+| 5 | Reviewed PR: [[#12048] Finish partial testcases #12742](https://github.com/TEAMMATES/teammates/pull/12742)|
+| 5 | Reviewed PR: [[#12048] Move getTypicalEntity functions to BaseTestCase #12744](https://github.com/TEAMMATES/teammates/pull/12744)|
+| 5 | Reviewed PR: [[#12048] Add test for getUsageStatisticsForTimeRange #12748](https://github.com/TEAMMATES/teammates/pull/12748)|
+| 6 | Reviewed PR: [[#12048] Remove typical data bundle from feedbackquestionlogic test #12750](https://github.com/TEAMMATES/teammates/pull/12750)|
+| 6 | Reviewed PR: [[#12048] Migrate SubmitFeedbackResponseAction #12720](https://github.com/TEAMMATES/teammates/pull/12720)|
+| 6 | Reviewed PR: [[#12048] Migrate FeedbackSessionOpeningSoonRemindersAction #12740](https://github.com/TEAMMATES/teammates/pull/12740)|
+| 6 | Reviewed PR: [[#12048] Migrate feedbackSessionPublishedRemindersAction #12741](https://github.com/TEAMMATES/teammates/pull/12741)|
+| 6 | Reviewed PR: [[#12048] Migrate FeedbackSessionClosingRemindersAction #12743](https://github.com/TEAMMATES/teammates/pull/12743)|
+| 6 | Contributed to Data Migration Scripts: [#12766](https://github.com/TEAMMATES/teammates/pull/12766), [#12767](https://github.com/TEAMMATES/teammates/pull/12767)|