Skip to content
This repository has been archived by the owner on Sep 22, 2022. It is now read-only.

Do not lowercase a blog handle #4

Closed
siman opened this issue May 6, 2020 · 2 comments
Closed

Do not lowercase a blog handle #4

siman opened this issue May 6, 2020 · 2 comments
Assignees
@F3Joule
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Milestone
M4 (current)

Comments

@siman
Copy link
Member

siman commented May 6, 2020

Do not lowercase a blog handle when storing it in a Blog struct. Lowercase only for storage item where we store a map of unique handles to resolve blog id.
@siman siman added good first issue Good for newcomers help wanted Extra attention is needed labels May 6, 2020
@subsocialdev subsocialdev added this to the M4 (current) milestone May 12, 2020
@F3Joule
Copy link
Member

F3Joule commented May 13, 2020

Fixed in F3Joule#8

@subsocialdev
Copy link

implemented and tested

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@F3Joule F3Joule

Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
M4 (current)
Development

No branches or pull requests
3 participants
@siman @F3Joule @subsocialdev