Skip to content

Commit

Permalink
Update jingting knowledge and progress
Browse files Browse the repository at this point in the history
  • Loading branch information
jingting1412 committed Apr 18, 2024
1 parent 9fde87a commit 7372644
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion students/jingting1412/knowledge.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ An interesting issue I've encountered this semester while researching on integra

Another crucial difference is that **CJS imports are synchronous** while **ESM imports are asynchronous**. As such, when importing ES modules into CJS, the normal `require('pagefind')` syntax would result in an error. Instead, you'll need to use `await import('pagefind')` to asynchronously import the module. This difference in imports is something that should be taken note of since we use both the ESM `import` syntax and CJS `require` syntax in various files in MarkBind.

### Resources
#### Resources
- [This blog post by Adam Coster](https://adamcoster.com/blog/commonjs-and-esm-importexport-compatibility-examples)
- [Typescript ESM/CJS Interoperability](https://www.typescriptlang.org/docs/handbook/modules/appendices/esm-cjs-interop.html)

Expand Down
3 changes: 0 additions & 3 deletions students/jingting1412/progress.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,12 @@ I mostly worked on the frontend of Markbind, improving components and the UI of
| Week | Achievements |
| ---- | ------------ |
| 2 | Authored PR: [Replacing default icon for conversion #2389](https://github.com/MarkBind/markbind/pull/2389) |
| 2 | Merged PR: [Replacing default icon for conversion #2389](https://github.com/MarkBind/markbind/pull/2389) |
| 3 | Authored PR: [MarkBind Template for Student Portfolio #2398](https://github.com/MarkBind/markbind/pull/2398) |
| 5 | Opened issue: [Set up deployment for student portfolio template #2418](https://github.com/MarkBind/markbind/issues/2418) |
| 5 | Discussed PR: [MarkBind Template for Student Portfolio #2385](https://github.com/MarkBind/markbind/issues/2385) |
| 6 | Authored PR: [Combining syntax cheat sheet and full syntax reference pages #2426](https://github.com/MarkBind/markbind/pull/2426) |
| Recess | Merged PR: [Combining syntax cheat sheet and full syntax reference pages #2426](https://github.com/jingting1412/markbind/commit/a44bdaa163ab457753e8e737711b2843dbcc9512) |
| 7 | Opened issue: [Add pageNav for Reader-Facing Features #2442](https://github.com/MarkBind/markbind/issues/2442) |
| 7 | Authored PR: [Add pageNav to Reader-Facing Features #2443](https://github.com/MarkBind/markbind/pull/2443) |
| 7 | Merged PR: [Add pageNav to Reader-Facing Features #2443](https://github.com/MarkBind/markbind/pull/2443) |
| 8 | Authored PR: [Docs: Upgrade to latest MarkBind version #12893](https://github.com/TEAMMATES/teammates/pull/12893) |
| 8 | Researched issue: [Auto-search content of pages #205](https://github.com/MarkBind/markbind/issues/205) |
| 9 | Discussed issue: [Missing output in Web3Forms section #2451](https://github.com/MarkBind/markbind/issues/2451) |
Expand Down

0 comments on commit 7372644

Please sign in to comment.