Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 2 Current »

\uD83D\uDDD3 Date

\uD83D\uDC65 Participants

Recording

Please be advised: Frontend Working Group meetings are recorded.

  • Video:

  • Chat:

  • Transcription:

\uD83D\uDDE3 Discussion topics

Time

Item

Presenter

Notes

21

Commit Lint failures with TypeScript aware repositories

Marlon Keating and ???

Marlon Keating is making his first appearance, representing edX!

edX is working on enabling TypeScript support in frontend build. They are running into some issues.

Mostly it's around testing, linting, and CI. There's a bug in commitlint. If there is no tsconfig, it will fail. If a downstream repo does not want to use TypeScript, they will not have a typescript config and the build will fail.

They would like more PRs on the workarounds for this because some of them are messy, and they'd like to know if anyone else has run into it so far.

https://github.com/conventional-changelog/commitlint/issues/3256

Example failure:
https://github.com/openedx/frontend-build/actions/runs/5730320499/job/15528860846?pr=430

Suggested resolution: Temporarily rename tsconfig file during commit lint stage, add comment with reason why and link to issue so anyone who is bothered by this workaround can fix it if upstream does.

4

New MFE for ORA

???

New MFE for ORA uses a potential pattern for complex XBlocks with an in-learning experience modal, with different views and with its own dependencies.

This functionality was initially built for LTI, and avoids nested iframes, using just one and leveraging the messaging functionality of the XBlock API. An ADR will be in Frontend-app-ORA.

5

react-query

???

Report of success and pleasure developing with react-query (apparently, part of TanStack Query) for state management with API calls.

✅ Action items

  •  

⤴ Decisions

  • Use the workaround for commit-lint suggested in the issue: Temporarily rename tsconfig before running.
  • No labels