Skip to content

Refactoring userDetails.jsx removes user config duplication#4161

Merged
benhalpern merged 2 commits intoforem:masterfrom
Aswathprabhu:refactor/refactoring_userdetails_removes_duplication
Oct 1, 2019
Merged

Refactoring userDetails.jsx removes user config duplication#4161
benhalpern merged 2 commits intoforem:masterfrom
Aswathprabhu:refactor/refactoring_userdetails_removes_duplication

Conversation

@Aswathprabhu
Copy link
Copy Markdown
Contributor

Refactoring userDetails.jsx, removes user config duplication

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

[optional] What gif best describes this PR or how it makes you feel?

alt_text

@pr-triage pr-triage Bot added the PR: unreviewed bot applied label for PR's with no review label Sep 30, 2019
Copy link
Copy Markdown
Contributor

@rhymes rhymes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it! Hashes overs ifs ;-)

@pr-triage pr-triage Bot added PR: reviewed-approved bot applied label for PR's where reviewer approves changes and removed PR: unreviewed bot applied label for PR's with no review labels Oct 1, 2019
@Aswathprabhu
Copy link
Copy Markdown
Contributor Author

Love it! Hashes overs ifs ;-)

@rhymes I always prefer that 😄

@pr-triage pr-triage Bot added PR: unreviewed bot applied label for PR's with no review and removed PR: reviewed-approved bot applied label for PR's where reviewer approves changes labels Oct 1, 2019
@benhalpern benhalpern merged commit eeb6cbd into forem:master Oct 1, 2019
@pr-triage pr-triage Bot added PR: merged bot applied label for PR's that are merged and removed PR: unreviewed bot applied label for PR's with no review labels Oct 1, 2019
redbassett pushed a commit to redbassett/dev.to that referenced this pull request Oct 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: merged bot applied label for PR's that are merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants