Skip to content

Update documentation to point to right hana module for JS #1133

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.

Sign up for GitHub

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 2 commits into from
Jul 23, 2024
Merged

Update documentation to point to right hana module for JS #1133

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
341ea48
use correct hana module
SamuelBrucksch Jul 23, 2024
9aa7438
update other references
SamuelBrucksch Jul 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions guides/databases.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ Following are cds-plugin packages for CAP Node.js runtime that support respectiv

| Database | Package | Remarks |
| ------------------------------ | ------------------------------------------------------------ | ---------------------------------- |
| **[SAP HANA Cloud](databases-hana)** | [`@sap/cds-hana`](https://www.npmjs.com/package/@sap/cds-hana) | recommended for production |
| **[SAP HANA Cloud](databases-hana)** | [`@cap-js/hana`](https://www.npmjs.com/package/@cap-js/hana) | recommended for production |
| **[SQLite](databases-sqlite)** | [`@cap-js/sqlite`](https://www.npmjs.com/package/@cap-js/sqlite) | recommended for development |
| **[PostgreSQL](databases-postgres)** | [`@cap-js/postgres`](https://www.npmjs.com/package/@cap-js/postgres) | maintained by community + CAP team |

Expand All @@ -46,13 +46,13 @@ npm add @cap-js/sqlite -D
Using SAP HANA for production:

```sh
npm add @sap/cds-hana
npm add @cap-js/hana
```

<!-- REVISIT: A bit confusing to prefer the non-copiable variant that doesn't get its own code fence -->
::: details Prefer `cds add hana` ...

... which also does the equivalent of `npm add @sap/cds-hana` but in addition cares for updating `mta.yaml` and other deployment resources as documented in the [deployment guide](deployment/to-cf#_1-using-sap-hana-database).
... which also does the equivalent of `npm add @cap-js/hana` but in addition cares for updating `mta.yaml` and other deployment resources as documented in the [deployment guide](deployment/to-cf#_1-using-sap-hana-database).

:::

Expand All @@ -66,7 +66,7 @@ The afore-mentioned packages use `cds-plugin` techniques to automatically config
"requires": {
"db": {
"[development]": { "kind": "sqlite", "impl": "@cap-js/sqlite", "credentials": { "url": "memory" } },
"[production]": { "kind": "hana", "impl": "@sap/cds-hana", "deploy-format": "hdbtable" }
"[production]": { "kind": "hana", "impl": "@cap-js/hana", "deploy-format": "hdbtable" }
}
}
}}
Expand Down