Skip to content

Nabrams docs add gel integration - #8000

Open
nabrams wants to merge 16 commits into
QwikDev:v1from
nabrams:nabrams-docs-add-gel-integration
Open

Nabrams docs add gel integration#8000
nabrams wants to merge 16 commits into
QwikDev:v1from
nabrams:nabrams-docs-add-gel-integration

Conversation

@nabrams

@nabrams nabrams commented Sep 25, 2025

Copy link
Copy Markdown
Member

What is it?

  • Docs / tests / types / typos

Description

I Have created a new integration for Qwik with GelDB(https://www.geldata.com/)

  • The integration automatically initializes the database with a sample schema, routes to view and modify the schema, and new scripts to utilize all of the new gel features.
  • The documentation has also been updated to guide users on how to install and get started with this new integration (View the documentation changes here -> https://nabrams-nabrams-docs-add-gel.qwik-8nx.pages.dev/docs/integrations/gel/)

Example repository with integration added: https://github.com/nabrams/qwik-app

screenshots of integration in a base app up and running

Screenshot 2025-10-10 at 1 37 20 PM Screenshot 2025-10-10 at 1 37 45 PM Screenshot 2025-10-10 at 1 37 29 PM Screenshot 2025-10-10 at 3 13 37 PM

View of the Gel UI database

Screenshot 2025-10-10 at 1 37 57 PM

Users directory page

Screenshot 2025-10-10 at 1 37 14 PM

Checklist

  • My code follows the developer guidelines of this project
  • I performed a self-review of my own code
  • I made corresponding changes to the Qwik docs
  • I added new tests to cover the fix / functionality

@changeset-bot

changeset-bot Bot commented Sep 25, 2025

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 3066524

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

github-actions Bot commented Sep 25, 2025

Copy link
Copy Markdown
Contributor
built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
qwik-docs ✅ Ready (View Log) Visit Preview 3066524

@nabrams
nabrams marked this pull request as ready for review October 15, 2025 17:20
@nabrams
nabrams requested review from a team as code owners October 15, 2025 17:20
@pkg-pr-new

pkg-pr-new Bot commented Nov 2, 2025

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@builder.io/qwik@8000
npm i https://pkg.pr.new/@builder.io/qwik-city@8000
npm i https://pkg.pr.new/eslint-plugin-qwik@8000
npm i https://pkg.pr.new/create-qwik@8000

commit: 3066524

@wmertens wmertens left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Some nitpicks, but in general I'm a bit hesitant to add so many files for a starter.

Is it not possible to create the starter with just some seed files and then to instruct the dev to run the generate command?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can you run pnpm api.update? Unfortunately in v1 this happens all the time.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

and this change should be reverted, it happens when running a local build

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

also revert this, prettier seems to sort it like this but order is important

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please revert this

@gioboa gioboa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @nabrams thanks for your help, is this PR still valid?

@wmertens
wmertens changed the base branch from main to v1 July 16, 2026 14:26
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.

4 participants