Skip to content

Login news #271

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Login news #271

wants to merge 5 commits into from

Conversation

xGuTeK
Copy link
Contributor

@xGuTeK xGuTeK commented Apr 14, 2025

Description

This pull request adds full support for sending and displaying dynamic server news using a new packet type: LS_NEWS. The system allows server administrators to manage news entries via the database, which are then sent to clients during the login phase.
Additionally, the refresh interval for reloading news from the database can be configured via the NewsRefreshInterval value in the Version.ini file.

⚠️ This PR also requires changes to the database, such as adding a new table for storing news entries.
    https://github.com/ko4life-net/ko-db/pull/40

obraz

💔 Thank you!

xGuTeK added 4 commits April 14, 2025 16:54
…ult INI values

Loads news data from a configured ODBC database table.
Implements periodic news refresh with a configurable interval (NewsRefreshInterval) from the INI file.
Ensures all required INI values are initialized with defaults if missing.
Parsing and handling of the LS_NEWS packet received from the login server.

News UI rendering and formatting.
@xGuTeK xGuTeK added the official_client_code_review Needs verification against the official client code using IDA/Ghidra label Apr 14, 2025
@UTengine
Copy link
Contributor

very nice addition we could set | as a linebreak here or we need to solve the pesky strings being cut off by n3ui

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
official_client_code_review Needs verification against the official client code using IDA/Ghidra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants