Skip to content

[Glense] Fix user ID types, add user tracking to chat, containerize video service, add comments API, remove hardcoded frontend data, improve UI#23

Merged
Brankonymous merged 5 commits intomainfrom
branko-grbic/fix-microservice-before-gluing-them-together
Mar 25, 2026
Merged

[Glense] Fix user ID types, add user tracking to chat, containerize video service, add comments API, remove hardcoded frontend data, improve UI#23
Brankonymous merged 5 commits intomainfrom
branko-grbic/fix-microservice-before-gluing-them-together

Conversation

@Brankonymous
Copy link
Copy Markdown
Owner

@Brankonymous Brankonymous commented Mar 24, 2026

Summary

  • Video Catalogue: changed all user ID fields from int to Guid to match Account service (was a hard blocker for inter-service communication)
  • Chat: added UserId (Guid) field to messages so we can track who sent what, populated from JWT claims
  • Added Video Catalogue service to docker-compose.yml (port 5002)
  • Fixed Chat service JWT secret to match Account service (tokens were being rejected)
  • Fixed frontend video API port from 5088 to 5002
  • Expanded seed data: 8 demo videos, 3 chat rooms with messages, sample donations between test users
  • Deleted stale EF migrations for both Video and Chat (incompatible with schema changes, using EnsureCreated for dev)
  • Added Comments feature to Video Catalogue service (model, controller, DTOs) with GET/POST/DELETE /api/videos/{videoId}/comments
  • Removed all hardcoded mock data from constants.jsx (videos, comments, videoInfo, chats arrays) — all data now comes from APIs
  • Moved all seed data (videos, comments, donations, users) into scripts/seed.sh so videos get real uploader IDs
  • Renamed seed-test-users.sh to seed.sh since it now seeds everything
  • Frontend components (VideoComments, ChannelDetail, VideoStream) now fetch from APIs instead of constants
  • Colored letter avatars everywhere: navbar, chat sidebar, chat messages, video comments, video channel bar
  • Fixed video API port 5088 → 5002 in frontend
  • Fixed Upload page: dark theme styling, removed manual "Uploader ID" field
  • Removed duplicate "Add to playlist" section from video page
  • Removed ChatList widget from feed sidebar (chat is accessible via navbar)
  • Video detail page fetches uploader profile to show real username

What was broken before

┌────────────────────────────────────────────────────┬──────────────────────────────────┐
│                      Problem                       │              Impact              │
├────────────────────────────────────────────────────┼──────────────────────────────────┤
│ Video service used int user IDs, Account uses Guid │ Cannot link videos to real users │
├────────────────────────────────────────────────────┼──────────────────────────────────┤
│ Chat messages had no user ID field                 │ Cannot show who sent a message   │
├────────────────────────────────────────────────────┼──────────────────────────────────┤
│ Chat JWT secret mismatched Account service         │ Auth tokens rejected by Chat     │
├────────────────────────────────────────────────────┼──────────────────────────────────┤
│ Video service not in docker-compose                │ Had to run manually              │
├────────────────────────────────────────────────────┼──────────────────────────────────┤
│ Frontend pointed to port 5088 for videos           │ Connection refused               │
└────────────────────────────────────────────────────┴──────────────────────────────────┘

Test plan

  • docker compose up --build starts all 4 services + databases
  • ./scripts/seed-test-users.sh creates users, wallets, and donations
  • Video service health check: curl http://localhost:5002/health
  • Chat service health check: curl http://localhost:5004/health
  • GET /api/videos returns 8 seeded videos with Guid uploaderIds
  • GET /api/chats returns 3 seeded chat rooms
  • Frontend feed page shows video thumbnails
  • Chat page shows chat rooms

@Brankonymous Brankonymous added this to the sprint-3 milestone Mar 24, 2026
@Brankonymous Brankonymous self-assigned this Mar 24, 2026
@Brankonymous Brankonymous moved this to Code Review in Glense Mar 24, 2026
@Brankonymous Brankonymous changed the title [Glense] Fix user ID types, add user tracking to chat, containerize video service [Glense] Fix user ID types, add user tracking to chat, containerize video service, add comments API, remove hardcoded frontend data, improve UI Mar 24, 2026
@Brankonymous Brankonymous merged commit 5b8e3a6 into main Mar 25, 2026
2 checks passed
@github-project-automation github-project-automation bot moved this from Code Review to Done in Glense Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants