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 Injector Component #109

Merged
merged 1 commit into from
Mar 28, 2019
Merged

Update Injector Component #109

merged 1 commit into from
Mar 28, 2019

Conversation

brh55
Copy link
Owner

@brh55 brh55 commented Mar 27, 2019

This should stop warnings associated with an invalid prop types #108

@brh55 brh55 merged commit 7fea177 into v0.5.0 Mar 28, 2019
brh55 added a commit that referenced this pull request Jun 4, 2019
* [In Progress] Adds Priority Option and Fixes Data Duplications (#86)

* Attempt to add in fixes suggested per other OSS modules

* Add semi colon

* First run for balance columns

* Rename function to findMinIndex

* Simplify array creation syntax

* Refactor balance

* Refactor priority and fix data update changes

* Update example to reference project dir

* Update masonry

* Minor stylistic changes, comment test

* Add updated snapshot

* Add lazy loading functionality (#87)

* Attempt to add in fixes suggested per other OSS modules

* Add semi colon

* First run for balance columns

* Rename function to findMinIndex

* Simplify array creation syntax

* Refactor balance

* Refactor priority and fix data update changes

* Update example to reference project dir

* Update masonry

* Minor stylistic changes, comment test

* Add updated snapshot

* Add append data

* Fix delay call reach

* Update snapshot for lazy load

* Set Alpha

* Add passable props to custom images (#96)

* Set alpha

* Revert

* passed data to customImageComponent (#94)

* Update package.json

* Fixed Bricks read only issue (#98)

I got an issue Bricks is read-only  because bricks is `const` at line 176 declared so it must be `let`

* Update package.json

* Update package.json (#109)
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