Skip to content

Refactor method in internal controller#7655

Merged
rhymes merged 1 commit intoforem:masterfrom
luchiago:luchiago/refactor-internal-config-controller-3739
May 7, 2020
Merged

Refactor method in internal controller#7655
rhymes merged 1 commit intoforem:masterfrom
luchiago:luchiago/refactor-internal-config-controller-3739

Conversation

@luchiago
Copy link
Copy Markdown
Contributor

@luchiago luchiago commented May 1, 2020

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

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Related Tickets & Documents

#3739

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

Added tests?

  • yes
  • no, because they aren't needed
  • no, because I need help

Added to documentation?

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

[optional] Are there any post deployment tasks we need to perform?

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

@luchiago luchiago requested a review from a team as a code owner May 1, 2020 20:48
@luchiago luchiago requested review from msarit and removed request for a team May 1, 2020 20:48
@pr-triage pr-triage Bot added the PR: unreviewed bot applied label for PR's with no review label May 1, 2020
Copy link
Copy Markdown
Contributor

@msarit msarit left a comment

Choose a reason for hiding this comment

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

Simple but powerful changes, thanks @luchiago! 👍

@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 May 6, 2020
@rhymes rhymes merged commit b2a6435 into forem:master May 7, 2020
@pr-triage pr-triage Bot added PR: merged bot applied label for PR's that are merged and removed PR: reviewed-approved bot applied label for PR's where reviewer approves changes labels May 7, 2020
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