Skip to content

Develop - #35

Merged
jhosepmyr merged 4 commits into
mainfrom
develop
Jul 10, 2026
Merged

Develop#35
jhosepmyr merged 4 commits into
mainfrom
develop

Conversation

@Eric396

@Eric396 Eric396 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Description

Feature module / area:

Related issue / US:


Type of Change

  • feat — new feature or UI component
  • fix — bug fix
  • refactor — code change without behavior change
  • test — tests only
  • docs — documentation only
  • build / ci — build, dependencies, or CI/CD
  • chore — maintenance

Checklist

  • The PR targets develop (not main)
  • Branch name follows feature/*, bugfix/*, or hotfix/*
  • Commits follow Conventional Commits
  • bun run lint passes locally (ESLint + angular-eslint)
  • bun run test passes locally (Vitest)
  • bun run build passes locally (no type errors, no budget exceeded)
  • New components use ChangeDetectionStrategy.OnPush and Angular signals
  • No localStorage/sessionStorage access for JWT tokens (use the auth store)
  • No bypassSecurityTrust* calls without explicit review
  • No secrets, credentials, or .env content committed
  • CHANGELOG.md updated under [Unreleased]

How to Test

Screenshots / recordings (if UI changes)

Notes (optional)

Eric396 and others added 4 commits July 10, 2026 13:03
fix: fix error related cto stopwatch of transcript
…eploy

The S3 sync marked every non-index asset immutable (max-age=1y), including the
non-fingerprinted i18n/{lang}.json files — so browsers cached them for a year
and never picked up translations added in later deploys (raw keys / stale copy
across devices). Exclude i18n from the immutable sync and upload it (like
index.html) with no-cache,must-revalidate so the browser revalidates it.
fix(deploy): serve i18n JSON no-cache so translations refresh after deploy
@Eric396
Eric396 requested a review from jhosepmyr as a code owner July 10, 2026 18:14
@jhosepmyr
jhosepmyr merged commit 68d3559 into main Jul 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants