Skip to content

Readme updates #9

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

Merged
merged 3 commits into from
Mar 4, 2025
Merged

Readme updates #9

merged 3 commits into from
Mar 4, 2025

Conversation

VirajAgarwal-couchbase
Copy link
Collaborator

This PR updates the README to provide clearer documentation and improve developer onboarding. The following enhancements have been made:

Changes:

  • Added a detailed explanation of the BaseConnection class to clarify its role in the framework.
  • Improved class and method documentation to explain advanced concepts without examples.
  • Included code snippets to support understanding of key methods.
  • Added the Contributing section

Merge pull request #7 from Couchbase-Ecosystem/main
Copy link

@nithishr nithishr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Has improved a lot. Just a few minor things & we should be good to go.

  • Double check that we use SQL++ instead of N1QL.

README.md Outdated

The **Couchbase-Streamlit Connector** provides a seamless way to integrate Couchbase with Streamlit applications. It simplifies database operations, allowing developers to interact with Couchbase clusters directly within Streamlit without requiring extensive SDK knowledge.

With this connector, developers can efficiently perform CRUD (Create, Read, Update, Delete) operations, execute N1QL (SQL++) queries, and dynamically manage Couchbase collections, scopes, and buckets—all within a Streamlit app. This enables rapid prototyping and interactive data visualization while leveraging Couchbase’s powerful database capabilities.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

execute SQL++ (formerly N1QL) queries

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed all mentions of N1QL to SQL++

README.md Outdated
With this connector, developers can efficiently perform CRUD (Create, Read, Update, Delete) operations, execute N1QL (SQL++) queries, and dynamically manage Couchbase collections, scopes, and buckets—all within a Streamlit app. This enables rapid prototyping and interactive data visualization while leveraging Couchbase’s powerful database capabilities.

**Key Benefits**
- **Simplified Database Access**: Eliminates the need for complex SDK implementations.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eliminates the need for separate SDK implementations.
We don't want to make the SDK implementation complex :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🫣 oops. Fixed the language here and made it 'seperate SDK implimentations'

README.md Outdated
**Key Benefits**
- **Simplified Database Access**: Eliminates the need for complex SDK implementations.
- **Streamlit-Native Integration**: Designed to work seamlessly with `st.connection()`.
- **Flexible Querying**: Supports both key-value operations and SQL-like queries using N1QL.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

N1QL to SQL++

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

README.md Outdated
pip install plotly geopy numpy
streamlit run src/Demo.py
```
Or access the hosted version: [Demo App](https://couchbase-connector-demo-app.streamlit.app/)

## Prerequisites
### System Requirements

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make it generic as Couchbase Server after you have verified it. Mention that the free tier of Capella is the easiest way to get started with using Couchbase with Streamlit.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated the Prereq (across all 3 readmes) to mention Couchbase Server compatibility as well

README.md Outdated
## Contributing
We welcome contributions to improve this project! Follow the guidelines below to ensure a smooth development process.

### Setting Up the Development Environment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While forking the clone URL will change to the user's Github account.
Might be better to link to the Github PR workflow docs like https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request

Add only the instructions to run the code for development that you have.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have added the link to the PR workflow. Not sure what to add to run the development code. Should I mention the tutorials for that? Also, should I remove the Reporting Issues sub-section?

README.md Outdated

Here are some helpful resources for working with Couchbase and Streamlit:
## Understanding `BaseConnection`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This section is not required as this is more in the Streamlit domain & not directly relevant for users of this integration. It is mainly relevant for developers & maintainers.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the section and just added some slight relevant concepts like abstract class in the CouchbaseConnector section in form of a few lines.

@VirajAgarwal-couchbase VirajAgarwal-couchbase merged commit 3dbac90 into production Mar 4, 2025
5 checks passed
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