Skip to content

Conversation

kvoli
Copy link
Contributor

@kvoli kvoli commented Jun 17, 2025

Null primary keys are not supported, however when a null primary key is present, browsing a selected type in the Hollow Explorer UI will fail with an OOB exception.

Handle null primary keys so that they are still displayed in the selected type page, with "NULL" substituted in-place of missing fields.

@kvoli kvoli self-assigned this Jun 17, 2025
@kvoli kvoli force-pushed the 250617.handle-null-pkey-explorer branch from 65e3103 to 80a730c Compare June 17, 2025 19:30
@kvoli kvoli marked this pull request as ready for review June 18, 2025 17:00
@kvoli kvoli requested a review from Sunjeet June 18, 2025 17:00
Null primary keys are not supported, however when a null primary key is
present, browsing a selected type in the Hollow Explorer UI will fail
with an OOB exception.

Handle null primary keys so that they are still displayed in the
selected type page, with "null" substituted in-place of missing fields.
@kvoli kvoli force-pushed the 250617.handle-null-pkey-explorer branch from 80a730c to ae2789d Compare June 19, 2025 15:31
@kvoli
Copy link
Contributor Author

kvoli commented Jun 19, 2025

Updated to display "null" instead of "NULL".

@kvoli
Copy link
Contributor Author

kvoli commented Jun 19, 2025

TYFTR!

@kvoli kvoli merged commit ad2c479 into master Jun 19, 2025
2 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.

2 participants