Staging #37

2026-09-30T17:39:20+02:00

aefril commented

2026-09-30 17:39:20 +02:00

No description provided.

aefril added 9 commits 2026-09-30 17:39:22 +02:00

Merge pull request 'Main' (#33 ) from main into staging 5e28b05770

Reviewed-on: #33

Reapply "feat(loyalty): EnakPoint & EnakCoin" (#32 ) c6062c4bb9

This reverts commit 4e24f9bbb0.

fix(deploy): require explicit environment and isolate containers per env 9b21af8892

Deploying staging from a folder checked out on main replaced the
production container, because the environment was inferred from the
current branch and both environments shared the container name
"apskel-pos".

- Environment is now a required argument (staging|production)
- Refuse to deploy when the branch doesn't match, HEAD is detached,
  or the tree is dirty
- Container name per environment; the legacy "apskel-pos" container
  is only removed by production deploys
- Abort if the port is held by another environment's container
- Confirmation prompt for production (--yes to skip)
- Fast-forward-only pull from origin/<branch>
- Keep the previous image and roll back automatically when the new
  container is not healthy; --rollback for manual rollback

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

refactor(loyalty): remove what is left of tokens c988a79d3b

Tokens are EnakCoin and no app uses the token names any more, so their
compatibility layer goes:

- GET /customer/tokens and its handler, service, processor and response
  types.
- total_tokens and tokens_history on GET /customer/wallet; last_updated
  now comes from the most recent row of either currency.
- token_used and tokens_remaining on game and spin responses, and
  sort_by=token_used on the game play list.
- TOKENS as a campaign type and reward type, with the mapping to COINS:
  migration 000092 already renamed the stored values.

The customer_tokens table and its entity stay, as cmd/wallet-migrate still
reads them, and LEGACY_TOKENS stays as the reference of the MIGRATION rows
it wrote. The docs list the removed names and their replacements.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Merge branch 'staging' into hotfix/deployment e200923b72

fix(loyalty): keep tokens removed after merging staging 35fe5703fd

Merging staging brought back, through its own re-apply of #32, the TOKENS
campaign mapping, its test and the token_used sort fallback that c988a79
had removed, because those hunks did not conflict. This restores the five
files to c988a79.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Merge pull request 'Hotfix/deployment' (#35 ) from hotfix/deployment into staging 609d434976

Reviewed-on: #35

fix(docker): healthcheck the port the app listens on 100e006218

The HEALTHCHECK curled localhost:3300/health, but the app listens on 4000
(server.port in infra/*.yaml, EXPOSE 4000). The check always failed, so
deployment.sh waited on 'Waiting for healthcheck...', found the container
unhealthy and rolled back to an image with the same broken check.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Merge pull request 'fix(docker): healthcheck the port the app listens on' (#36 ) from fix/dockerfile-healthcheck-port into staging e8b31035f7

Reviewed-on: #36

aefril merged commit e94eb2c26d into main

2026-09-30 17:39:29 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: apksel-dev/apskel-pos-backend#37