Skip to content

Server side getColPolygonHeight fix #3946

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

Merged
merged 8 commits into from
Jan 10, 2025

Conversation

tederis
Copy link
Member

@tederis tederis commented Jan 10, 2025

Fixes the return type of server side getColPolygonHeight. Currently it returns a table, whereas two floats are expected.
Additional info: #3537 (comment)

@tederis tederis added the bugfix Solution to a bug of any kind label Jan 10, 2025
@TheNormalnij TheNormalnij added the backwards-incompatible Should be merged after the release of 1.6.1 label Jan 10, 2025
@TheNormalnij TheNormalnij added this to the 1.6.1 milestone Jan 10, 2025
@TheNormalnij
Copy link
Member

Nice catch!
Btw. You need to sync your master with mta-blue master and force-push it to your repo

@tederis tederis force-pushed the GetColPolygonHeight_fix branch from 3f3520c to 7a29cac Compare January 10, 2025 12:52
@botder botder merged commit 0a6b23b into multitheftauto:master Jan 10, 2025
6 checks passed
MTABot pushed a commit that referenced this pull request Jan 10, 2025
0a6b23b Fix return values of getColPolygonHeight on server-side (PR #3946)
018a706 Update client en_US pot
@botder
Copy link
Member

botder commented Jan 10, 2025

I made a mistake here, should've merged this after 1.6.1 release.

@botder botder modified the milestones: 1.6.1, Backlog Jan 10, 2025
botder added a commit that referenced this pull request Jan 10, 2025
botder added a commit that referenced this pull request Jan 10, 2025
@botder botder modified the milestones: Backlog, Next Release (1.6.2) Jan 10, 2025
MTABot pushed a commit that referenced this pull request Jan 10, 2025
f3b3013 Revert "Fix return values of getColPolygonHeight on server-side (PR #3946)"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backwards-incompatible Should be merged after the release of 1.6.1 bugfix Solution to a bug of any kind
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants