guides: add pushSetup for ingestion API #4126
Conversation
✔️ Code generated!
📊 Benchmark resultsBenchmarks performed on the method using a mock server, the results might not reflect the real-world performance.
|
shortcuts
changed the title
| "pushSetup": { | ||
| "method": "pushTask", | ||
| "parameters": { | ||
| "taskID": "YOUR_TASK_ID", |
There was a problem hiding this comment.
I hope this will be replaced by the correct taskID on the dashboard ! cc @morganleroi
shortcuts
force-pushed
the
feat/guides-for-push
branch
from
649d298 to
af6612a
Compare
| // special lambda for dynamic templates | ||
| bundle.put("dynamicTemplate", new DynamicTemplateLambda(this)); | ||
| bundle.put("dynamicSnippet", new DynamicSnippetLambda(this, models, operations, language, client)); | ||
| if (new File("tests/CTS/guides/" + client + ".json").exists()) { |
There was a problem hiding this comment.
should this be done inside of the lambda directly to avoid leaking the internal logic here ?
There was a problem hiding this comment.
since it's always called I wanted to avoid no-op as it's less explicit when looking at this file, idk what you prefer
There was a problem hiding this comment.
I think this files makes it harder since now the path is in multiple files rather than just the lambda, this file should be more generic IMO
| } | ||
|
|
||
| // use runID in the Observability debugger | ||
| fmt.Println("run", run.RunID) |
| {{> snippets/import}} | ||
|
|
||
| using Action = Algolia.Search.Models.Ingestion.Action; | ||
| using PushTaskRecords = Algolia.Search.Models.Ingestion.PushTaskRecords; |
There was a problem hiding this comment.
| using PushTaskRecords = Algolia.Search.Models.Ingestion.PushTaskRecords; | |
| using Algolia.Search.Models.Ingestion.PushTaskRecords; |
There was a problem hiding this comment.
this aliases automatically? nice
Co-authored-by: Clément Vannicatte <[email protected]> Co-authored-by: Pierre Millot <[email protected]>
🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-3199
Changes included:
add a simple push snippet for the end of the setup on the dashboard