Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Please use a delimiter between org/team names #43

Merged
merged 2 commits into from May 23, 2020
Merged

Please use a delimiter between org/team names #43

merged 2 commits into from May 23, 2020

Conversation

ghost
Copy link

@ghost ghost commented May 22, 2020

Please use a delimiter between org/team names in the log.

Expected:
Log messages will not be hard to distinguish due to org/team names not being represented as discreet units.

Actual:
Org/team names are combined into a single string which must be manually separated.

Reproducing:
Log into nexus via Github oauth using an account with multiple teams in an organization (e.g org1/team1 and org1/team2). Observe that in the log the organization and team names are combined (e.g. org1/team1org1/team2).

Please use a delimiter between org/team names in the log.

Expected:
Log messages will not be hard to distinguish due to org/team names not being represented as discreet units.

Actual:
Org/team names are combined into a single string which must be manually separated.

Reproducing:
Log into nexus via Github oauth using an account with multiple teams in an organization (e.g org1/team1 and org1/team2). Observe that in the log the organization and team names are combined (e.g. org1/team1org1/team2).
@tobilarscheid
Copy link
Member

Very helpful improvement! Thank you!

@tobilarscheid tobilarscheid merged commit 6593943 into L21s:master May 23, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant