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

Update fts-supported-queries-date-range.adoc #3292

Open
wants to merge 1 commit into
base: release/7.2
Choose a base branch
from

Conversation

jon-strabala
Copy link
Contributor

The documentation was completely missing for this complex feature/syntax - this is valid for all 7.X version and most likely 6.X versions of the documentation.

This is a recurring problem no one (our own staff) knows how to make Custom DateTime Parsers becuase they don't know that the templates have to be in Golang syntax.

@sarahlwelton
Copy link
Contributor

sarahlwelton commented Jul 25, 2023

Jon, isn't this satisfied by: https://docs.couchbase.com/cloud/search/create-custom-date-time-parser.html

There is equivalent in Server, I just need to fix the ToC.

And if we're going to be phasing out this documentation with the ToC updates, I don't see a need to update this old file.

@jon-strabala
Copy link
Contributor Author

jon-strabala commented Jul 25, 2023

Jon, isn't this satisfied by: https://docs.couchbase.com/cloud/search/create-custom-date-time-parser.html

There is equivalent in Server, I just need to fix the ToC.

I just saw this ... as such I want to adjust the PR as "create-custom-date-time-parser" just doesn't give enough information. No one want's to spend time digging in other offsite documents - this burned both Ankit and myself last weekend.

So I want to update both files a bit (oh there is a third file) it all should be consistent at a minimum.

@sarahlwelton
Copy link
Contributor

No, we don't document 3rd party because it's subject to change and we can't be held responsible for maintaining documentation on the Go package. I would recommend that you don't make too many changes, here.

@jon-strabala
Copy link
Contributor Author

Understood, however one issue the 3rd party go site does not' document our default dateTimeOptions (date parser) IMHO what this is and what it covers should be documented.

I will think about things a bit more what I propose will be minimal

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