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

Simplified api #336

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Simplified api #336

wants to merge 1 commit into from

Conversation

alpha0010
Copy link
Contributor

The current JS api, though full of options, can be overly complicated for basic usage. I propose including this simplified API (draft not complete) in the package. It can be used by:

[...]
import sqlite from 'react-native-sqlite-storage/simple';

class Foo extends Component<Props, State> {
  [...]
  async logSomeData() {
    const db = await sqlite.open('bar.sqlite');
    const user = await db.get('SELECT * FROM users WHERE id = ?', [42]);
    console.log(user);
  }
}

Is there interest in this new api?

@andpor
Copy link
Owner

andpor commented Apr 24, 2019

Perhaps you should open an issue first and start a discussion? Perhaps others will voice their opinion...

@mitevdev
Copy link

@andpor OK, an issue has been opened and it has been commented as well. What's the next step?

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.

3 participants