fix sonar mobile scope and coverage - #13
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 84007f9725
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| const headers: Record<string, string> = { | ||
| ...(options.headers ?? {}), | ||
| } | ||
| const headers: Record<string, string> = options.headers ?? {} |
There was a problem hiding this comment.
apiClient now aliases options.headers directly and then mutates it by adding auth/content-type fields, which introduces hidden side effects for callers that reuse a headers object across requests. In that case, one JSON request can permanently inject Content-Type: application/json into the shared object and break later FormData uploads (wrong boundary/content type), so this should keep copying headers before mutation as it did previously.
Useful? React with 👍 / 👎.
| <div | ||
| role="button" | ||
| tabIndex={0} | ||
| className={`${articleClassName} text-left w-full`} |
There was a problem hiding this comment.
This element is still focusable and handles Enter/Space, but removing role="button" strips the control semantics that assistive technologies rely on to announce it as an actionable UI element. Keyboard users can still activate it, but screen-reader users will no longer get proper button affordance for the primary card action, so this is an accessibility regression unless the container is converted to a native <button>.
Useful? React with 👍 / 👎.
…olish (#8, #13) (#326) Year-skip arrows on month nav (both platforms). Month-stats redesigned as a data-driven extensible StatTile grid. Web: view switcher (Month/Week/Range) + Google-Calendar-style time-grid (hourly rows, day columns, timed-habit blocks, all-day band, now-line) for week + custom contiguous-range; mobile keeps month. Sync screen: Sincronizar agora->Sincronizar, deselect-all text->icon button. web+mobile parity for shared parts; grids web-only by design; both locales. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 participant
No description provided.