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

Already on GitHub? Sign in to your account

New AssetSync Class #3

Merged
merged 2 commits into from Mar 3, 2014

Conversation

Projects
None yet
2 participants
Contributor

davydotcom commented Mar 3, 2014

Working on new assetSync class that can sync more than one provider (if using to sync to multiple regions). Also leverages new ContentType logic and localStorageProvider. This may not be merge ready yet as I havent integrated into the gant scripts yet but thought you might want to have a look.

Contributor

davydotcom commented Mar 3, 2014

Ok I haven't tested this at all, but the AssetSync class is no longer dependent on s3 specific classes. Supports iterating over multiple providers. I did remove some gant code that should no longer be needed.

@benorama benorama added a commit that referenced this pull request Mar 3, 2014

@benorama benorama Merge pull request #3 from davydotcom/patch-1
New AssetSync Class + Refactoring
4feb946

@benorama benorama merged commit 4feb946 into agorapulse:master Mar 3, 2014

Owner

benorama commented Mar 3, 2014

OK, thanks !
I'll have a look on this tonight.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment