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

feat: support forever-loading CustomDataProvider #11

Merged
merged 1 commit into from
May 27, 2019
Merged

Conversation

amcgee
Copy link
Member

@amcgee amcgee commented May 27, 2019

This adds support for a forever-loading state in CustomDataProvider, which is useful for i.e. Storybook demos. Usage as follows:

<CustomDataProvider options={{ loadForever: true }}>
   <...>
</CustomDataProvider>

@amcgee amcgee requested a review from varl May 27, 2019 10:17
@amcgee
Copy link
Member Author

amcgee commented May 27, 2019

This is technically a feature but might not warrant a minor version bump? How semantic are we?

@varl
Copy link
Contributor

varl commented May 27, 2019

This is technically a feature but might not warrant a minor version bump? How semantic are we?

Pragmatically increasingly semantic.

More leeway to "sneaking" things in before they become widespread. When things stabilize and use is up, we are more strict.

@amcgee
Copy link
Member Author

amcgee commented May 27, 2019

Pragmatically increasingly semantic.

Tubular.

@varl
Copy link
Contributor

varl commented May 27, 2019

LGTM 🙆‍♂

@amcgee
Copy link
Member Author

amcgee commented May 27, 2019

Shall I bump the minor version with a feat: prefix then? Probably doesn't matter since we pin versions anyway

@varl
Copy link
Contributor

varl commented May 27, 2019

Sure, go for feat:.

@amcgee amcgee merged commit e80c7b7 into master May 27, 2019
@amcgee amcgee deleted the feat/load-forever branch May 27, 2019 11:03
dhis2-bot pushed a commit that referenced this pull request May 27, 2019
# [1.2.0](v1.1.0...v1.2.0) (2019-05-27)

### Features

* support forever-loading CustomDataProvider ([#11](#11)) ([e80c7b7](e80c7b7))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 1.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants