Add info to date processor docs
#127434
Add info to date processor docs
#127434
Conversation
PeteGillinElastic
added
v9.1.0
>docs
PeteGillinElastic
force-pushed
the
date-processor-docs
branch
from
cbc8f5d to
59eab0b
Compare
PeteGillinElastic
force-pushed
the
date-processor-docs
branch
from
2f34a95 to
a1502b4
Compare
elasticsearchmachine
added
Team:Data Management
|
Pinging @elastic/es-docs (Team:Docs) |
|
Pinging @elastic/es-data-management (Team:Data Management) |
| @@ -25,6 +24,14 @@ $$$date-options$$$ | |||
| | `on_failure` | no | - | Handle failures for the processor. See [Handling pipeline failures](docs-content://manage-data/ingest/transform-enrich/ingest-pipelines.md#handling-pipeline-failures). | | |||
| | `tag` | no | - | Identifier for the processor. Useful for debugging and metrics. | | |||
|
|
|||
| The `timezone` option may have two effects on the behavior of the processor: | |||
There was a problem hiding this comment.
Idea: If you gave this section a heading you could link to it from the table instead of writing (see below)
Looking at the URL preview, there are no subheadings at all on this page which makes it hard to scan :)
There was a problem hiding this comment.
Oh, that's a good idea. I've just added a bunch of section headings, and turned both the belows into links. (I previewed the docs locally and the cross-references work as expected, so hopefully I got the format right.)
I've still left it saying 'see below', because I needed something as an anchor text. If there's a preference for a different style, please let me know.
There was a problem hiding this comment.
Nice! Made a suggestion to incorporate link into the text more naturally :)
There was a problem hiding this comment.
Thanks, yeah, that's better. Done. (I somehow mashed the wrong button in github and failed to apply your suggestion, but I've pushed a commit with the exact same change!)
This does two things:
timezoneoption actually does. The existing wording is misleading.PST. This has come up at least twice recently.