• Joined on 2026-02-23
maximus commented on pull request maximus/simpl-liste#64 2026-04-09 01:22:55 +00:00
fix: issues #60 #61 #62 #63 — inbox, refresh, subtask depth, chevron/detail

PR Review — fix: issues #60 #61 #62 #63

Verdict: APPROVE ✅

Les quatre corrections sont cohérentes, bien séparées, et couvrent les trois couches (mobile, web, API) quand…

maximus pushed to issue-60-fix-duplicate-inbox at maximus/simpl-liste 2026-04-09 01:16:28 +00:00
78471543c3 fix: separate subtask expand chevron from detail view icon (#63)
maximus pushed to issue-60-fix-duplicate-inbox at maximus/simpl-liste 2026-04-09 01:13:52 +00:00
21020406b2 fix: prevent sub-subtask creation, limit nesting to 2 levels (#62)
maximus pushed to issue-60-fix-duplicate-inbox at maximus/simpl-liste 2026-04-09 01:05:03 +00:00
894ac03072 feat: add refresh button on web + swipe-to-refresh on mobile (#61)
maximus commented on pull request maximus/simpl-liste#64 2026-04-09 00:57:42 +00:00
fix: issues #60 #61 #62 #63 — inbox, refresh, subtask depth, chevron/detail

PR Review: fix: resolve duplicate inbox on web after mobile sync (#60)

Verdict: APPROVE

Summary

The sync endpoint now correctly handles the case where a mobile client pushes an inbox…

maximus pushed to issue-60-fix-duplicate-inbox at maximus/simpl-liste 2026-04-09 00:55:06 +00:00
6c36ebcce5 fix: wrap inbox merge in transaction, revert seed to random UUID (#60)
maximus commented on pull request maximus/simpl-liste#64 2026-04-09 00:52:00 +00:00
fix: issues #60 #61 #62 #63 — inbox, refresh, subtask depth, chevron/detail

Review — PR #64

Verdict: ✅ APPROVE

Le fix est correct et bien ciblé. La logique de déduplication d'inbox lors du sync est solide : vérifier l'existence, réassigner les tâches,…

maximus created pull request maximus/simpl-liste#64 2026-04-09 00:50:28 +00:00
fix: resolve duplicate inbox on web after mobile sync (#60)
maximus pushed to issue-60-fix-duplicate-inbox at maximus/simpl-liste 2026-04-09 00:50:19 +00:00
d9daf9eda4 fix: resolve duplicate inbox on web after mobile sync (#60)
maximus created branch issue-60-fix-duplicate-inbox in maximus/simpl-liste 2026-04-09 00:50:18 +00:00
maximus opened issue maximus/simpl-liste#63 2026-04-09 00:47:41 +00:00
fix: separate subtask expand chevron from detail view icon
maximus opened issue maximus/simpl-liste#62 2026-04-09 00:47:28 +00:00
fix: prevent sub-subtask creation (limit nesting to 2 levels)
maximus opened issue maximus/simpl-liste#61 2026-04-09 00:47:21 +00:00
feat: add refresh button on web + swipe-to-refresh on mobile
maximus opened issue maximus/simpl-liste#60 2026-04-09 00:47:19 +00:00
fix: resolve duplicate inbox on web after mobile sync
maximus opened issue maximus/simpl-liste#59 2026-04-09 00:38:09 +00:00
sync - bogue refresh
maximus released v1.5.2 at maximus/simpl-liste 2026-04-08 21:35:20 +00:00
maximus pushed tag v1.5.2 to maximus/simpl-liste 2026-04-08 19:42:25 +00:00
maximus pushed to master at maximus/simpl-liste 2026-04-08 19:42:23 +00:00
71ee702739 chore: bump version to 1.5.2 (versionCode 11)
maximus commented on issue maximus/simpl-liste#51 2026-04-08 19:33:55 +00:00
Intégrer @logto/rn avec expo-secure-store sur mobile

L'intégration Logto est déjà complète dans le code actuel :

  • ✅ @logto/rn v1.1.0 + expo-secure-store installés
  • ✅ logtoConfig.ts avec endpoint, appId sl-mobile-native, redirect…
maximus closed issue maximus/simpl-liste#51 2026-04-08 19:33:55 +00:00
Intégrer @logto/rn avec expo-secure-store sur mobile