Skip to content

Commit

Permalink
Merge branch 'master' of https://github.com/nus-cs3281/2025
Browse files Browse the repository at this point in the history
  • Loading branch information
@CYX22222003
CYX22222003 committed Feb 22, 2025
2 parents cdac607 + f9ce5f8 commit bc58763
Show file tree
Hide file tree
Showing 5 changed files with 49 additions and 11 deletions.
11 changes: 7 additions & 4 deletions students/DhiraPT/progress.md
View file
Original file line number Diff line number Diff line change
@@ -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) |
16 changes: 13 additions & 3 deletions students/jasonqiu212/progress.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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) |
23 changes: 19 additions & 4 deletions students/joeng03/knowledge.md
View file
Original file line number Diff line number Diff line change
@@ -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.

...
### 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)
4 changes: 4 additions & 0 deletions students/yucongkoo/progress.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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) |
6 changes: 6 additions & 0 deletions students/yyccbb/progress.md
View file
Original file line number Diff line number Diff line change
@@ -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

0 comments on commit bc58763

Please # to comment.