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

docs: EXPOSED-626 fix nullTransform example code #2292

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

kdhyo
Copy link
Contributor

@kdhyo kdhyo commented Nov 1, 2024

Description

Fixes a bug in the example code for nullTransform provided in the documentation.

Detailed description:

  • What:
    Fix Docs Example code
  • Why:
    If you try to use the example code as is, you'll get a compiler error.
  • How:
    Change transform() to nullTransform().

Type of Change

Please mark the relevant options with an "X":

  • Bug fix
  • New feature
  • Documentation update

Updates/remove existing public API methods:

  • Is breaking change

Affected databases:

  • MariaDB
  • Mysql5
  • Mysql8
  • Oracle
  • Postgres
  • SqlServer
  • H2
  • SQLite

Checklist

  • Unit tests are in place
  • The build is green (including the Detekt check)
  • All public methods affected by my PR has up to date API docs
  • Documentation for my change is up to date

Related Issues

@bog-walk bog-walk self-assigned this Nov 1, 2024
@bog-walk bog-walk self-requested a review November 1, 2024 14:05
Copy link
Member

@bog-walk bog-walk left a comment

Choose a reason for hiding this comment

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

Thanks for starting this PR @kdhyo

Please see suggested changes in YouTrack issue and please apply them to the documentation-website file directly, instead of this .html file.

@kdhyo kdhyo requested a review from bog-walk November 1, 2024 16:18
@bog-walk
Copy link
Member

bog-walk commented Nov 4, 2024

Thanks for the changes @kdhyo
All website and api docs are auto-synced with each release (aimed for the end of each month). So your changes will become visible then.

@bog-walk bog-walk merged commit 3725287 into JetBrains:main Nov 4, 2024
3 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