avery42/delfin
9
146
Fork
You've already forked delfin
28

Only show quick connect button if server has quick connect enabled #168

Merged
avery42 merged 2 commits from quick-connect-check-enabled into main 2024年11月29日 22:25:25 +01:00

Forgot about this before merging the last PR.

Also did some refactoring.

Forgot about this before merging the last PR. Also did some refactoring.
Create QuickConnectClient to reuse reqwest client when polling
All checks were successful
ci/woodpecker/push/release Pipeline was successful
ci/woodpecker/pr/release Pipeline was successful
ci/woodpecker/push/test Pipeline was successful
ci/woodpecker/push/lint Pipeline was successful
ci/woodpecker/pr/test Pipeline was successful
ci/woodpecker/pr/lint Pipeline was successful
ci/woodpecker/pull_request_closed/release Pipeline was successful
ci/woodpecker/pull_request_closed/test Pipeline was successful
ci/woodpecker/pull_request_closed/lint Pipeline was successful
36f6026262 
Also did some refactoring to quick connect polling
avery42 scheduled this pull request to auto merge when all checks succeed 2024年11月29日 22:13:36 +01:00
Sign in to join this conversation.
No reviewers
Labels
Clear labels
Kind/Breaking
Breaking change that won't be backward compatible
Kind/Bug
Something is not working
Kind/Documentation
Documentation changes
Kind/Enhancement
Improve existing functionality
Kind/Feature
New functionality
Kind/Security
This is security issue
Kind/Testing
Issue or pull request related to testing
Needs Design
Packaging/AUR
Issue with the AUR package
Packaging/Flatpak
Issue with the Flatpak
Priority
Critical
 The priority is critical
Priority
High
 The priority is high
Priority
Low
 The priority is low
Priority
Medium
 The priority is medium
Reviewed
Confirmed
 Issue has been confirmed
Reviewed
Duplicate
 This issue or pull request already exists
Reviewed
Invalid
 Invalid issue
Reviewed
Upstream
 Issue with a package or library we use that can't be fixed independently
Reviewed
Won't Fix
 This issue won't be fixed
Status
Abandoned
 Somebody has started to work on this but abandoned work
Status
Blocked
 Something is blocking this issue or pull request
Status
Need More Info
 Feedback is required to reproduce issue or to continue work
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
avery42/delfin!168
Reference in a new issue
avery42/delfin
No description provided.
Delete branch "quick-connect-check-enabled"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?