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

Add customdriver method to Persist module #249

Merged
merged 1 commit into from
May 22, 2020
Merged

Conversation

fohletex
Copy link
Contributor

@fohletex fohletex commented Jan 6, 2020

I recently ran into issues, where I wanted to use ForerunnerDB, but had no access to either window.Storage nor IndexedDB, WebSQL and localStorage. I further didn't want to use NodePersist. Hence, some research brought me to the localforage#defineDriver-API. Thus, I extended the Persist module with a customdriver method, which allows you to import a custom localforage driver for any other storage destinations, than the available ones.

@fohletex
Copy link
Contributor Author

fohletex commented Jan 6, 2020

hmmm ... testbear fails by travis CI - on my machine it perfectly passes .... - seems as if there is a version conflict in the setup configuration - could that be?

                    _     _
                   ( \---/ )   
                    ( . . )
             ___,--.(__Y__),--.___
            |   `--'       `--'   |
            | Test Bear is HAPPY! |
            |    _,-.     ,-._    |
            |___(ooO )___( Ooo)___|
                 `--'     `--' 

EDIT: Saw, that the error is already documented: #246

@Irrelon
Copy link
Owner

Irrelon commented May 22, 2020

If testbear is happy, I am happy :)

@Irrelon
Copy link
Owner

Irrelon commented May 22, 2020

Actually that's some nice work there, thanks for documenting as well!

@Irrelon Irrelon merged commit e894a0e into Irrelon:master May 22, 2020
@Irrelon
Copy link
Owner

Irrelon commented May 22, 2020

Merged!

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