Skip to content

Set default agent after registration#3213

Merged
infirit merged 2 commits into
blueman-project:mainfrom
infirit:defaultagentonreply
Jun 2, 2026
Merged

Set default agent after registration#3213
infirit merged 2 commits into
blueman-project:mainfrom
infirit:defaultagentonreply

Conversation

@infirit

@infirit infirit commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

#3209 rightfully points out we could potentially ask to be default agent before we finished registration. But instead of making it synchronous keep the registration asynchronous and request default agent when dbus replies.

@sonarqubecloud

Copy link
Copy Markdown

infirit added 2 commits June 1, 2026 23:05
This avoids the possibility we request default agent before an Agent is
registered as reported by blueman-project#3209. But unlike blueman-project#3209 register is async.
Before it would show "register_agent: Register Agent" which is kinda
useless as it just repeats the method name.

Also "promote" this to debug as this is mostly useful for debugging.
@infirit infirit force-pushed the defaultagentonreply branch from 692822f to a401819 Compare June 1, 2026 21:05
@sonarqubecloud

sonarqubecloud Bot commented Jun 1, 2026

Copy link
Copy Markdown

@infirit infirit enabled auto-merge (rebase) June 1, 2026 21:06
@infirit infirit requested a review from cschramm June 1, 2026 21:09
@infirit infirit merged commit a5fc2f0 into blueman-project:main Jun 2, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants