Skip to content
This repository has been archived by the owner on May 5, 2021. It is now read-only.

Assemble Hooks #47

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Assemble Hooks #47

wants to merge 2 commits into from

Conversation

camdagr8
Copy link

@camdagr8 camdagr8 commented Feb 16, 2017

Often is the case where I need to alter the assemble process and/or contents. I'm able to do this by adding in hooks to the assembler at certain points (similar to how helpers are added):

  1. parseLayouts()
  2. parseLayoutIncludes()
  3. parseData()
  4. parseMaterials()
  5. parseViews()
  6. parseDocs()
  7. After assemble is completed (using the assembly object values).

I've documented the hooks and commented inline where they are executed.

@camdagr8
Copy link
Author

@LukeAskew have you had a chance to take a look at this?

@LukeAskew
Copy link
Member

This totally makes sense.

Sorry I've not been able to devote attention to this project for a while, but I just sat down to start work on a v2. It will likely involve a rewrite of the engine, but I think this is a concept I should include. I will for sure give you credit if this makes it in.

I don't know that it makes sense to fold this in right now, but I'll leave this open in case that changes.

Thanks!

@camdagr8
Copy link
Author

camdagr8 commented Apr 6, 2017

Cool, yeah, we ended up rolling something new based off of this.

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