Skip to content

feat: character list window#602

Merged
majestyotbr merged 3 commits intoopentibiabr:mainfrom
majestyotbr:characterlist
Oct 27, 2023
Merged

feat: character list window#602
majestyotbr merged 3 commits intoopentibiabr:mainfrom
majestyotbr:characterlist

Conversation

@majestyotbr
Copy link
Contributor

@majestyotbr majestyotbr commented Oct 19, 2023

characterlist

@majestyotbr majestyotbr marked this pull request as draft October 19, 2023 21:20
@majestyotbr majestyotbr marked this pull request as ready for review October 19, 2023 22:43
@luanluciano93

This comment was marked as resolved.

@majestyotbr majestyotbr force-pushed the characterlist branch 3 times, most recently from 3622823 to ce952ce Compare October 20, 2023 17:40
@Nottinghster

This comment has been minimized.

@marcosvf132
Copy link
Contributor

marcosvf132 commented Oct 23, 2023

It should work with all client versions, not working with 860

image

Not, it shouldnt. The structure of the login on 12+ client is completely different from the 11- clients.
The QT client (12+) get the characterlist and initial messages throught HTTP POST, using the login.php from your AAC.
The old client (11-) get the characterlist from the server protocollogin.

If you want this feature to work on all clients version perfectly you must change your protocollogin structure to include characters appearances.

By default the only data that comes from the server protocollogin is the character name, server name, server IP, game Port and premium days.

@opentibiabr opentibiabr deleted a comment from Vysius Oct 23, 2023
@majestyotbr

This comment was marked as resolved.

@majestyotbr majestyotbr marked this pull request as draft October 23, 2023 10:25
@Nottinghster

This comment was marked as resolved.

@majestyotbr

This comment was marked as resolved.

@Nottinghster

This comment was marked as resolved.

@majestyotbr

This comment was marked as resolved.

@majestyotbr

This comment was marked as resolved.

@Nottinghster

This comment was marked as resolved.

@majestyotbr majestyotbr marked this pull request as ready for review October 25, 2023 06:19
@majestyotbr majestyotbr merged commit 23e0626 into opentibiabr:main Oct 27, 2023
@majestyotbr majestyotbr deleted the characterlist branch October 27, 2023 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants