docs: update TESTING.md with UI/UX fixes verification
This commit is contained in:
@@ -138,7 +138,31 @@ STACK_REGISTRY_ID=bKDYM5X7NN34x_lRDjWbz # Registry ID for Docker auth
|
||||
1. **Registry URL format** - Use `git.app.flexinit.nl`, NOT `https://git.app.flexinit.nl`
|
||||
2. **Username in image path** - Must be `oussamadouhou`, not `odouhou`
|
||||
3. **Dokploy URL** - Must use public URL for container-to-container communication
|
||||
4. **Health check** - SSL errors treated as "alive" during cert provisioning
|
||||
4. **Health check** - Now uses Dokploy API status check (not HTTP fetch)
|
||||
5. **Resource limits** - Removed temporarily (Dokploy API format issues with CPU/memory values)
|
||||
6. **Typewriter race condition** - Fixed by tracking active instances and canceling previous
|
||||
|
||||
### UI/UX Fixes (2026-01-10 v2)
|
||||
|
||||
| Issue | Fix | Status |
|
||||
|-------|-----|--------|
|
||||
| Double letters in typewriter | Track active instances, cancel before starting new | ✅ Verified |
|
||||
| Flag emojis not showing | Replaced with text labels (NL/AR/EN) | ✅ Verified |
|
||||
| SSE disconnects during deployment | Use Dokploy API status instead of HTTP health check | ✅ Verified |
|
||||
| 'yourname' not translated | Added translation key per language | ✅ Verified |
|
||||
|
||||
### Health Check Change
|
||||
|
||||
The health check was changed from HTTP fetch (which failed from inside Docker) to Dokploy API status check:
|
||||
|
||||
```typescript
|
||||
// Before: HTTP fetch (failed - container can't reach external URL)
|
||||
const response = await fetch(`${state.url}/`);
|
||||
|
||||
// After: Dokploy API status check (works)
|
||||
const app = await this.client.getApplication(applicationId);
|
||||
if (app.applicationStatus === 'done') { /* success */ }
|
||||
```
|
||||
|
||||
### Test Commands
|
||||
|
||||
|
||||
Reference in New Issue
Block a user