diff --git a/students/DhiraPT/progress.md b/students/DhiraPT/progress.md index 8cbed881..26f12ea5 100644 --- a/students/DhiraPT/progress.md +++ b/students/DhiraPT/progress.md @@ -1,4 +1,7 @@ -| Week | Achievements | -|------|----------------------------------------------------------| -| 2 | Submitted Issue: [Duplicate Feedback Sessions Appear on Student Page #13199](https://github.com/TEAMMATES/teammates/issues/13199) | -| 4 | Reviewed PR: [[#12048] Migrate tests for GetAccountActionTest](https://github.com/TEAMMATES/teammates/pull/13212) | +| Week | Achievements | +|------|----------------------------------------------------------------------------------------------------------------------------------------------| +| 2 | Submitted Issue: [Duplicate Feedback Sessions Appear on Student Page #13199](https://github.com/TEAMMATES/teammates/issues/13199) | +| 4 | Reviewed PR: [[#12048] Migrate tests for GetAccountActionTest #13212](https://github.com/TEAMMATES/teammates/pull/13212) | +| 5 | Submitted Issue: [Standardize the Naming Convention for FeedbackSession Actions #13228](https://github.com/TEAMMATES/teammates/issues/13228) | +| 6 | Merged PR: [[#12048] Migrate Tests for DeleteStudentActionTest #13204](https://github.com/TEAMMATES/teammates/pull/13204) | +| 6 | Merged PR: [[#12048] Migrate Tests for DeleteStudentsActionTest #13205](https://github.com/TEAMMATES/teammates/pull/13205) | diff --git a/students/jasonqiu212/progress.md b/students/jasonqiu212/progress.md index da24dca2..ba8e855a 100644 --- a/students/jasonqiu212/progress.md +++ b/students/jasonqiu212/progress.md @@ -50,12 +50,22 @@ | 5 | Reviewed PR: [[#12048] Migrate tests for UpdateFeedbackQuestionActionTest #13231](https://github.com/TEAMMATES/teammates/pull/13231) | | 5 | Reviewed PR: [[#13104] Add mappings #13166](https://github.com/TEAMMATES/teammates/pull/13166) | | 5 | Closed PR: [Test date #13198](https://github.com/TEAMMATES/teammates/pull/13198) | +| 5 | Reviewed PR: [[#13164] Fix typo #13165](https://github.com/TEAMMATES/teammates/pull/13165) | +| 5 | Discussed issue: [Failing component test #13173](https://github.com/TEAMMATES/teammates/issues/13173) | +| 6 | Reviewed PR: [[#8557] Standardize locale #13162](https://github.com/TEAMMATES/teammates/pull/13162) | +| 6 | Reviewed PR: [[#12048] Migrate tests for GetAccountsAction.java #13214](https://github.com/TEAMMATES/teammates/pull/13214) | +| 6 | Reviewed PR: [[#12048] Migrate tests for GetAccountActionTest #13212](https://github.com/TEAMMATES/teammates/pull/13212) | +| 6 | Reviewed PR: [[#12048] Migrate tests #13206](https://github.com/TEAMMATES/teammates/pull/13206) | +| 6 | Reviewed PR: [[#12048] Migrate Tests for DeleteStudentActionTest #13204](https://github.com/TEAMMATES/teammates/pull/13204) | +| 6 | Reviewed PR: [[#12048] Migrate Tests for DeleteStudentsActionTest #13205](https://github.com/TEAMMATES/teammates/pull/13205) | +| 6 | Reviewed PR: [[#13160] Update documentation to use Docker Compose v2 #13161](https://github.com/TEAMMATES/teammates/pull/13161) | +| 6 | Closed PR: [[#13202] Feature/show collapse all button #13225](https://github.com/TEAMMATES/teammates/pull/13225) | ## External Project Work | Week | Project | Achievements | | ---- | ------------------------------------------------------------------------- | --------------------------------------------------------------------------------------------------------- | -| 2 | [reactjs/zh-hans.react.dev](https://github.com/reactjs/zh-hans.react.dev) | Merged PR: [Improve translation](https://github.com/reactjs/zh-hans.react.dev/pull/1656) | -| 5 | [reactjs/react.dev](https://github.com/reactjs/react.dev) | Discussed issue: [link to createClass is broken](https://github.com/reactjs/react.dev/issues/7476) | +| 2 | [reactjs/zh-hans.react.dev](https://github.com/reactjs/zh-hans.react.dev) | Merged PR: [Improve translation #1656](https://github.com/reactjs/zh-hans.react.dev/pull/1656) | +| 5 | [reactjs/react.dev](https://github.com/reactjs/react.dev) | Discussed issue: [link to createClass is broken #7476](https://github.com/reactjs/react.dev/issues/7476) | | 5 | [reactjs/react.dev](https://github.com/reactjs/react.dev) | Raised issue: [[Typo]: Misspelling on Setup Page #7606](https://github.com/reactjs/react.dev/issues/7606) | -| 5 | [reactjs/react.dev](https://github.com/reactjs/react.dev) | Raised PR: [Fix setup page #7607](https://github.com/reactjs/react.dev/pull/7607) | +| 5 | [reactjs/react.dev](https://github.com/reactjs/react.dev) | Merged PR: [Fix setup page #7607](https://github.com/reactjs/react.dev/pull/7607) | diff --git a/students/joeng03/knowledge.md b/students/joeng03/knowledge.md index a258f742..859e7929 100644 --- a/students/joeng03/knowledge.md +++ b/students/joeng03/knowledge.md @@ -1,7 +1,22 @@ -### Tool/Technology 1 +### ESLint +Learnt about how ESLint ensures a unified style of JS/TS code. +Had the chance to go through the ESLint documentation for +member-delimiter-style, +https://eslint.style/rules/ts/member-delimiter-style, +understand how it works, and make the modifications in the ESLint +configurations and the codebase to ensure CI job for lintFrontend passes. -List the aspects you learned, and the resources you used to learn them, and a brief summary of each resource. +### Vite +Learnt about how Vite build identifies the base directory when +serving static assets. -### Tool/Technology 2 +### Vercel +Learnt how to configure Vercel on a GitHub repository. -... \ No newline at end of file +### Immutability in Java +Learnt about the various aspects to consider when designing +and immutable class in Java, such as: +- private and final variables +- elimination of setter methods +- returning copies for mutable variables +- considerations for constructor design (method overloading vs Builder pattern) \ No newline at end of file diff --git a/students/yucongkoo/progress.md b/students/yucongkoo/progress.md index 7b396469..bb678510 100644 --- a/students/yucongkoo/progress.md +++ b/students/yucongkoo/progress.md @@ -14,3 +14,7 @@ | 5 | Merged PR: [Update github-actions.yml to run when push/PR #1322](https://github.com/CATcher-org/CATcher/pull/1322) | | 5 | Reviewed PR: [Fix incorrect Github REST API response types #1321](https://github.com/CATcher-org/CATcher/pull/1321#discussion_r1948430106) | | 5 | Reviewed PR: [https://github.com/CATcher-org/CATcher/pull/1319#pullrequestreview-2604841262](https://github.com/CATcher-org/CATcher/pull/1319#pullrequestreview-2604841262) | +| 6 | Submitted Bug: [Line through UI for issues-deleted.component.css is not working on Safari #1327](https://github.com/CATcher-org/CATcher/issues/1327) | +| 6 | Reviewed PR: [Fix strikethrough on Safari #1328](https://github.com/CATcher-org/CATcher/pull/1328#pullrequestreview-2619991780) | +| 6 | Merged PR: [Remove feature branch from github-actions.yml #1329](https://github.com/CATcher-org/CATcher/pull/1329) | +| 6 | Reviewed PR: [Merge Feature bug trimming branch to main #1331](https://github.com/CATcher-org/CATcher/pull/1331#pullrequestreview-2620080929) | diff --git a/students/yyccbb/progress.md b/students/yyccbb/progress.md index e69de29b..2c3ab18f 100644 --- a/students/yyccbb/progress.md +++ b/students/yyccbb/progress.md @@ -0,0 +1,6 @@ +| Week | Achievements +| ---- | ------------------------------------- +| 5 | Authored issues #2602, #2603 +| 5 | Explored debugger configurations for JetBrains IDEs and VSCode +| 5 | Explored Markdown-it codebase +| 5 | Deployed local LLM and fed aggregated MarkBind codebase to it