University-of-Basel-Studentprojects/Programmierprojekt
Archived
1
0
Fork 0
Code Issues 17 Pull Requests Actions Wiki Activity

Add serverside LIST_USERS command #63

New Issue
Closed
opened 2026-04-09 18:15:00 +02:00 by lars.winzer · 14 comments
lars.winzer commented 2026-04-09 18:15:00 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

Task

Summary

Add command LIST_USERS to list all users currently active on the server.

Context

Request, CommandParser, Response and CommandHandler

Current Progress

  • ListUsersRequest
  • ListUsersParser
  • ListUsersResponse
  • ListUsersHandler
  • Update protocol documentation
  • Write JavaDoc

Notes

One pain-point could be response size, with tens or hundrets or active users on the server.

But introducing pagination only complicates things and is far from realistic anyways.

Request

LIST_USERS

Response

Response for two active users: Bob and Max Mustermann.

+OK
  USERS
    USER
      USERNAME=Bob
      ID=f7db024e-31f4-4edf-8368-ee0f696c0ea9
    END
    USER
      USERNAME='Max Mustermann'
      ID=14e33ef8-c1f8-4c9f-84fc-470b65f7d405
    END
  END
END

Checklist

  • I outlined checkpoints describing phases of my work related to this task
  • I added relevant labels
  • I linked to other issues or branches that must be completed first
## Task <!--The reccommended type is: Task--> ### Summary Add command `LIST_USERS` to list all users currently active on the server. ### Context Request, CommandParser, Response and CommandHandler ### Current Progress - [x] ListUsersRequest - [x] ListUsersParser - [x] ListUsersResponse - [x] ListUsersHandler - [x] Update protocol documentation - [x] Write JavaDoc ### Notes One pain-point could be response size, with tens or hundrets or active users on the server. But introducing pagination only complicates things and is far from realistic anyways. #### Request ``` LIST_USERS ``` #### Response Response for two active users: Bob and Max Mustermann. ``` +OK USERS USER USERNAME=Bob ID=f7db024e-31f4-4edf-8368-ee0f696c0ea9 END USER USERNAME='Max Mustermann' ID=14e33ef8-c1f8-4c9f-84fc-470b65f7d405 END END END ``` ### Checklist - [x] I outlined checkpoints describing phases of my work related to this task - [x] I added relevant labels - [x] I linked to other issues or branches that must be completed first
lars.winzer commented 2026-04-09 18:15:07 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

assigned to @lars.winzer

assigned to @lars.winzer
lars.winzer commented 2026-04-09 18:15:19 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

changed title from Add serverside LIST_USERS to Add serverside LIST_USERS command

<p>changed title from <code class="idiff">Add serverside LIST_USERS</code> to <code class="idiff">Add serverside LIST_USERS<span class="idiff left right addition"> command</span></code></p>
lars.winzer commented 2026-04-09 18:15:59 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

created branch feat/63-add-serverside-list-users-command to address this issue

created branch [`feat/63-add-serverside-list-users-command`](/cs108-fs26/Gruppe-13/-/compare/main...feat%2F63-add-serverside-list-users-command) to address this issue
lars.winzer commented 2026-04-09 18:18:48 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

marked the checklist item ListUsersRequest as completed

marked the checklist item **ListUsersRequest** as completed
lars.winzer commented 2026-04-09 18:18:49 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

marked the checklist item ListUsersParser as completed

marked the checklist item **ListUsersParser** as completed
lars.winzer commented 2026-04-09 18:30:51 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

marked the checklist item ListUsersResponse as completed

marked the checklist item **ListUsersResponse** as completed
lars.winzer commented 2026-04-09 18:47:56 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

marked the checklist item ListUsersHandler as completed

marked the checklist item **ListUsersHandler** as completed
lars.winzer commented 2026-04-09 18:59:27 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

marked the checklist item Update protocol documentation as completed

marked the checklist item **Update protocol documentation** as completed
lars.winzer commented 2026-04-09 18:59:40 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

changed the description

changed the description
lars.winzer commented 2026-04-09 19:00:25 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

changed the description

changed the description
lars.winzer commented 2026-04-09 19:07:15 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

marked the checklist item Write JavaDoc as completed

marked the checklist item **Write JavaDoc** as completed
lars.winzer commented 2026-04-09 19:08:09 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

added 45m of time spent at 2026-04-09 12:00:00 +0200

added 45m of time spent at 2026-04-09 12:00:00 +0200
lars.winzer commented 2026-04-09 19:11:32 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

mentioned in merge request !90

mentioned in merge request !90
lars.winzer commented 2026-04-09 19:13:18 +02:00 (Migrated from git.scicore.unibas.ch)
Copy Link
Copy Source

mentioned in commit 74497a1e8e

mentioned in commit 74497a1e8e11db0264642cb9d266cfa51086f6d8
lars.winzer (Migrated from git.scicore.unibas.ch) closed this issue 2026-04-09 19:13:19 +02:00
This repo is archived. You cannot comment on issues.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
achievement
Tasks related to milestone achievements
 bug
Issues related to programming errors on both client and server
 ci
Issues related to problems with the gitlab ci (incorrect or missing jobs)
 client
Issues (To-Dos and tasks) related to the client
 commands
Issues and tasks related to commands and thier infrastructure (CommandHandler, CommandParser, ...)
 documentation
Issues related to the documentation (for new or unclear, incorrect or incomplete entries)
 enhancement
Issues related to enhancements (also known as feature-requests)
 game-engine
Issues related to the game engine (incorrect state transitions or move validations)
 linting
Issues related to linting i.e. false detections, too generous linting, ...
 messages
Issues and tasks related to the messages and chats
 network
Issues and tasks related to the network (SessionReader / SessionWriter, Parser, ResponseBody, ...)
 outreach
Issues and tasks related to the outreach website and everything directly related to it
 server
Issues (To-Dos and tasks) related to the server
 task
Issues outlining and discussing a general task or implementation
 templates
Issues related to GitLab templates
 ui
Issues and tasks related to the client-side ui
 unit-test
Issues related to missing or already planned unit tests
 user
Issues and tasks related to the user management
No Label commands server task
Milestone
No items
No Milestone Milestone 4
Assignees
Clear assignees
lars-winzer (Lars Winzer)
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: University-of-Basel-Studentprojects/Programmierprojekt#63
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?