Skip to content

Conversation

@vishalnarkhede
Copy link
Contributor

@vishalnarkhede vishalnarkhede commented Jul 18, 2020

Submit a pull request

CLA

  • I have signed the Stream CLA (required).
  • The code changes follow best practices
  • Code changes are tested (add some information if not applicable)

Description of the pull request

  • Grouping the components and moving them to their respective directories
  • Moving MessageSystem from MessageSimple to MessageList (pending testing)

@vishalnarkhede vishalnarkhede marked this pull request as ready for review July 18, 2020 08:56
@vishalnarkhede vishalnarkhede changed the title Directory restructuring of components CRNS-105 - Directory restructuring of components Jul 18, 2020
@vishalnarkhede vishalnarkhede requested a review from mahboubii July 20, 2020 15:27
Copy link
Contributor

@mahboubii mahboubii left a comment

Choose a reason for hiding this comment

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

looks good 👍 , the only issue I see here is in git history some files are deleted and recreated under different path which makes the file lose its commit history.

@vishalnarkhede vishalnarkhede merged commit 52754bb into master Jul 22, 2020
@delete-merged-branch delete-merged-branch bot deleted the vishal/directory-restructuring branch July 22, 2020 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants