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

Update module-factories.md #1785

Closed
wants to merge 1 commit into from
Closed

Update module-factories.md #1785

wants to merge 1 commit into from

Conversation

dear-lizhihua
Copy link
Collaborator

image

@skipjack
Copy link
Collaborator

skipjack commented Jan 14, 2018

@dear-lizhihua that page actually does belong in the "Plugins" section though I see how it can be confusing. Note that I do have #1754 open that refactors and standardizes the whole plugins section which should make things more clear. I took out the pages that mostly contained TODOs but will re-add some of those pages like this one later on (as the factories are tapable).

I'll make sure that it's clear why this page is in the "Plugins" section and maybe rename it to "Factories" to prevent confusion. I'm hoping to make the final push on that branch this week.

@skipjack skipjack closed this Jan 14, 2018
@sokra sokra deleted the patch branch October 10, 2020 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants