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
Projects
M4, M5
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

@F3Joule F3Joule added this to Review in progress in M4, M5 May 13, 2020
@F3Joule F3Joule added this to Review in progress in Backlog May 13, 2020
@subsocialdev subsocialdev removed this from Review in progress in Backlog May 28, 2020
@subsocialdev
Copy link

implemented and tested

M4, M5 automation moved this from Review in progress to Done Jun 16, 2020
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
No open projects
M4, M5
  
Done
Milestone
M4 (current)
Development

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