Skip to content

UI - Add password fill compatiblity for "connectToServer" (2.19+)#1726

Merged
PabstMirror merged 2 commits intomasterfrom
ui/password-autocomplete-219
Feb 14, 2025
Merged

UI - Add password fill compatiblity for "connectToServer" (2.19+)#1726
PabstMirror merged 2 commits intomasterfrom
ui/password-autocomplete-219

Conversation

@veteran29
Copy link
Copy Markdown
Member

@veteran29 veteran29 commented Jan 9, 2025

When merged this pull request will:

https://github.com/CBATeam/CBA_A3/blob/master/addons/ui/fnc_initDisplayPassword.sqf#L6
`_targetServer = _display getVariable "guid"; // 127.0.0.1:2302 always the resolved IP address, not the domain`
Starting with next week dev, and profiling branch v20
The variable is set before the init event, it should always be present

`diag_log [_display, _display getVariable ["guid", "?"]];`
`11:39:47 [Display #64,"164.152.123.222:2331"]`

PabstMirror
PabstMirror reviewed Jan 10, 2025
View reviewed changes
@veteran29 @johnb432
Update addons/ui/fnc_initDisplayPassword.sqf
1152336 
Co-authored-by: johnb432 <58661205+johnb432@users.noreply.github.com>
@PabstMirror PabstMirror added this to the 3.18.2 milestone Feb 14, 2025
@PabstMirror PabstMirror merged commit 27612af into master Feb 14, 2025
4 checks passed
@PabstMirror PabstMirror deleted the ui/password-autocomplete-219 branch February 14, 2025 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PabstMirror PabstMirror PabstMirror approved these changes

+2 more reviewers

@dedmen dedmen dedmen left review comments

@johnb432 johnb432 johnb432 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Enhancement

Projects

None yet

Milestone

3.18.2

Development

Successfully merging this pull request may close these issues.

4 participants

@veteran29 @dedmen @PabstMirror @johnb432