Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor Feature/Module/connector => "Plugin" (Second try) #591

Closed
wants to merge 4 commits into from

Conversation

aneilbaboo
Copy link

See discussion at #565

Renamed modules folders to plugins
Renamed connector.js to plugin.js
Renamed Feature class to Plugin
Renamed modules variables to plugins
Renamed misc other Module___.___ files to Plugin___.___

Aneil Mallavarapu and others added 4 commits December 23, 2017 10:30
Step one of the refactor: renamed the files to preserve history.
Note that tests of this commit will fail.

* rename src/client/modules=>src/client/plugins
* rename src/client/modules/connector.js=>src/client/plugins/plugin.js
* rename src/client/modules/connector.web.js=>src/client/plugins/plugin.web.js
* rename src/server/modules=>src/server/plugins
* rename src/server/modules/connector.js=>src/server/plugins/plugin.js
* rename plug folders and files in tools/templates
To improve clarity and distinguish from renaming other connector.js imports to plugin.js
  * Renamed Feature class to Plugin
  * Renamed module[s] variables to plugin[s]
Refactor Feature/Module/connector => Plugin
@aneilbaboo
Copy link
Author

Still some problems here. Will fix and reissue a new PR.

@aneilbaboo aneilbaboo closed this Dec 23, 2017
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.

1 participant