Skip to content
This repository has been archived by the owner on Aug 28, 2019. It is now read-only.

proposed new definition of Functional managers in index.md #7857

Merged
merged 4 commits into from
Nov 25, 2017
Merged

proposed new definition of Functional managers in index.md #7857

merged 4 commits into from
Nov 25, 2017

Conversation

rwoj
Copy link
Contributor

@rwoj rwoj commented Nov 15, 2017

New proposition of Functional Managers definition. 😊


✅️ By submitting this PR, I have verified the following

  • Added descriptive name to PR
    • Your PR should NOT be called Update index.md since it does not help the maintainer understand what has been changed.
    • For example, if you create a Variables article inside the Python directory, the pull request title should be Python: add Variables article.
    • Other examples are Git: edit Git Commit article or PHP: create PHP section and add Data Structures article
  • [-] Reviewed necessary formatting guidelines in CONTRIBUTING.md.
  • No plagiarized, duplicate or repetitive content that has been directly copied from another source.

Copy link
Contributor

@nicleary nicleary left a comment

Choose a reason for hiding this comment

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

The article content looks good to me, except there seem to be a few grammatical errors. For instance, the first line should be "A functional manager is..." and in the second paragraph the wording "in more agile organizations different models exist" would probably be better wording.
The content seems good however! Thanks for your contribution!

@rwoj
Copy link
Contributor Author

rwoj commented Nov 25, 2017

would you please let me know if something is expected that I will do with that proposed article, before it is published?

@nicleary
Copy link
Contributor

@rwoj Thank you for fixing this! Looks good to me. I'll merge it now.
Thank you for your contribution!

@nicleary nicleary merged commit d3ed081 into freeCodeCamp:master Nov 25, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants