Skip to content
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

Work/macos use local socket #9214

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

erikjv
Copy link
Collaborator

@erikjv erikjv commented Nov 12, 2021

No description provided.

Previously, the socket api prefix was created by concatenating
SOCKETAPI_TEAM_IDENTIFIER_PREFIX and OC_APPLICATION_REV_DOMAIN together,
so the first variable had to end in a "." to get the correct result.
This change moves adding that period from configuration to the places
where the concatenation happens.
@erikjv
Copy link
Collaborator Author

erikjv commented Nov 17, 2021

Merge after #9212 is merged.

@erikjv erikjv changed the title Draft: Work/macos use local socket Work/macos use local socket Nov 17, 2021
@erikjv erikjv added the macOS label Nov 17, 2021
@erikjv erikjv force-pushed the work/macos-use-local-socket branch 2 times, most recently from 8765d5e to 2f6bf2f Compare November 19, 2021 07:39
@sonarcloud
Copy link

sonarcloud bot commented Nov 19, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@github-actions github-actions bot added the Stale label Mar 20, 2022
@github-actions github-actions bot closed this Mar 27, 2022
@TheOneRing TheOneRing removed the Stale label Mar 27, 2022
@TheOneRing TheOneRing reopened this Mar 27, 2022
@github-actions github-actions bot added the Stale label Apr 27, 2022
@github-actions github-actions bot closed this May 5, 2022
@TheOneRing TheOneRing removed the Stale label May 5, 2022
@TheOneRing TheOneRing reopened this May 5, 2022
@TheOneRing TheOneRing added this to the 2.12 milestone May 5, 2022
@TheOneRing TheOneRing removed this from the Desktop Client 4.0 milestone Apr 20, 2023
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.

2 participants