Skip to content
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

Document t_first as SUCCESS response. #65

Merged
merged 2 commits into from
Jul 2, 2024
Merged

Document t_first as SUCCESS response. #65

merged 2 commits into from
Jul 2, 2024

Conversation

stefano-ottolenghi
Copy link
Contributor

@stefano-ottolenghi stefano-ottolenghi commented Jul 1, 2024

@robsdedude and I discovered that t_first was undocumented. I took the change to clean up the response metadata as well and alphabetically sort the entries. I copied from the t_last entry, but I'm unsure about the version labels @dotStart

Clean up `SUCCESS` response metadata and sort alphabetically.
@robsdedude
Copy link
Member

robsdedude commented Jul 1, 2024

I copied from the t_last entry, but I'm unsure about the version labels.

According to the Python driver's code, they've been renamed together going from Bolt 2 to Bolt 3.

@neo-technology-commit-status-publisher
Copy link

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

Copy link
Member

@robsdedude robsdedude left a comment

Choose a reason for hiding this comment

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

⏱️

@stefano-ottolenghi stefano-ottolenghi merged commit d07e582 into main Jul 2, 2024
5 checks passed
@stefano-ottolenghi stefano-ottolenghi deleted the t_first branch July 2, 2024 07:23
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.

None yet

3 participants