Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs - filters #26220

Merged
merged 11 commits into from
Nov 9, 2022
Merged

docs - filters #26220

merged 11 commits into from
Nov 9, 2022

Conversation

nllho
Copy link
Contributor

@nllho nllho commented Nov 2, 2022

Super gnarly topic... so many dependencies!

Main changes:

  • Filters troubleshooting: consolidated info with docs.
  • Data reference: moved out of SQL section and brought up to date (it's a great resource for data types, which is the root cause of many issues, filters or otherwise).
  • SQL: updated info on field filters and troubleshooting.

These changes break some links in Learn. I've got them noted down -- will follow up when the 45 docs go out.

Gonna take one more pass just to look at formatting tomorrow, but otherwise this is ready for review.

@nllho nllho added Type:Documentation backport Automatically create PR on current release branch on merge labels Nov 2, 2022
@nllho nllho requested a review from jeff-bruemmer November 2, 2022 21:13
@nllho nllho merged commit 0337a42 into master Nov 9, 2022
@nllho nllho deleted the docs-filters branch November 9, 2022 16:50
github-actions bot pushed a commit that referenced this pull request Nov 9, 2022
metabase-bot bot added a commit that referenced this pull request Nov 9, 2022
Co-authored-by: Natalie <nat@metabase.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
backport Automatically create PR on current release branch on merge Type:Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants