Skip to content
This repository has been archived by the owner on Nov 24, 2021. It is now read-only.

Enable store file name different from model file name #32

Merged
merged 2 commits into from
Jan 29, 2016

Conversation

gfpacheco
Copy link
Contributor

I'm building an app that needs multiples stores with the same data model. This optional init parameter enables such case.

@3lvis
Copy link
Owner

3lvis commented Jan 28, 2016

Hi @gfpacheco,

This looks good, I'll look into adding a few unit tests to ensure it doesn't break in the future.

3lvis added a commit that referenced this pull request Jan 29, 2016
Enable store file name different from model file name
@3lvis 3lvis merged commit 84df866 into 3lvis:master Jan 29, 2016
@gfpacheco gfpacheco deleted the store-path branch January 29, 2016 12:01
@3lvis
Copy link
Owner

3lvis commented Feb 8, 2016

Hey! Do you have an example on how are you using this? I'm having a hard time to come up with a use case.

@gfpacheco
Copy link
Contributor Author

Well, in my case I build apps for events. Usually there's one app for each event. There's this customer that wants one app for some of his events. Then I built another db scheme for the shared part of the app and each event keeps using the exact same scheme. When the user selects one event, I open the event's store and bring the details for that one.

@3lvis
Copy link
Owner

3lvis commented Feb 8, 2016

What's different between the modelName and the storeName, then? Seems like they are the same.

@3lvis
Copy link
Owner

3lvis commented Feb 8, 2016

Oh, wait, I got it now. Thanks 😊

@gfpacheco
Copy link
Contributor Author

Hahah, just to clarify for anyone who get here: model name is the database scheme file name, store name is the name of the sqlite store file. I can have multiples stores using the same db scheme =]

@3lvis
Copy link
Owner

3lvis commented Feb 8, 2016

Released on 4.1.0

🌟 🌟 Thanks 👏 🌟 🌟

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

Successfully merging this pull request may close these issues.

None yet

2 participants