Skip to content

Commit b3cc768

Browse files
authored
Merge pull request #182 from grafbase/tomhoule-lzmwytpsplzn
grafbase-postgres: lookup fields should not have a non-null wrapper
2 parents 0490270 + 9926359 commit b3cc768

File tree

8 files changed

+89
-85
lines changed

8 files changed

+89
-85
lines changed

Cargo.lock

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

cli/postgres/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,10 @@
22

33
All notable changes to the Grafbase PostgreSQL CLI tool will be documented in this file.
44

5+
## [0.3.9] - 2025-09-05
6+
7+
- Generate `@lookup` fields without non-null wrappers, since per the composite schemas spec, lookup fields should be nullable.
8+
59
## [0.3.7] - 2025-05-22
610

711
### Added

cli/postgres/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[package]
22
name = "grafbase-postgres"
3-
version = "0.3.8"
3+
version = "0.3.9"
44
edition = "2024"
55
license = "Apache-2.0"
66

crates/postgres-introspection/src/render/query.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ pub fn render<'a>(
101101
None => format!("{}_lookup", table.client_name()).to_camel_case(),
102102
};
103103

104-
let mut field = Field::new(field_name, format!("[{}]!", table.client_name()));
104+
let mut field = Field::new(field_name, format!("[{}]", table.client_name()));
105105

106106
field.set_description(format!(
107107
"Lookup multiple {} for subgraph joins",

0 commit comments

Comments
 (0)