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

Rdh resource fetcher events #150

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

Conversation

kimtuck
Copy link

@kimtuck kimtuck commented Nov 22, 2016

This pull request is Finalized

Related issue(s) and/or pull request(s)

See also #144*

Test cases, sample files

Additional information

Provides additional events to be emitted when resource fetcher is getting internal resources (files listed in the HTML of the chapter) and for files embedded in the css file(s).

The Bibliotheca hybrid app uses this to help it's resource fetcher plugin batch up file requests. Essentially, when the start event is emitted, it begins a batch; then individual file requests are queued up. When the end event is emitted, it sends the queued up list of files to the API, which reads the contents of each specified file and returns the contents as a response.

This helps to make the resource fetcher more performant because it limits the number of requests sent across the wire.

danielweck and others added 2 commits September 20, 2016 17:54
…after doing so. This provides plugin resource fetchers a hook to do some additional processing when fetching internal resources. For example, a resource fetcher might batch up the requested files, and process them all in one go.
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.

2 participants