Skip to content

Conversation

@heatlikeheatwave
Copy link
Contributor

Changes

  • Added collectCodeListItems helper function, which is required to add the codeListItems property found in Algolia records (see screenshot below)
Screenshot 2025-01-07 at 17 30 56
  • 1:1 with this function in mktg-content-workflows

  • Added tests for the function

Asana task

@heatlikeheatwave heatlikeheatwave self-assigned this Jan 7, 2025
@heatlikeheatwave heatlikeheatwave requested a review from a team as a code owner January 7, 2025 22:33
@heatlikeheatwave heatlikeheatwave requested review from EnMod and RubenSandwich and removed request for a team January 7, 2025 22:33
@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Jan 8, 2025, 4:32 PM
Unified Docs API ✅ Ready (Inspect) Visit Preview Jan 8, 2025, 4:16 PM

@heatlikeheatwave heatlikeheatwave requested review from LeahMarieBush, aaronvanderlip and rmainwork and removed request for EnMod January 7, 2025 22:33
Copy link
Contributor

@RubenSandwich RubenSandwich left a comment

Choose a reason for hiding this comment

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

Looks good. 👍🏻

Copy link
Contributor

@LeahMarieBush LeahMarieBush left a comment

Choose a reason for hiding this comment

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

LGTM!

@heatlikeheatwave heatlikeheatwave merged commit cf74014 into main Jan 8, 2025
3 checks passed
@heatlikeheatwave heatlikeheatwave deleted the heat/feat/algolia/collect-code-list-items branch January 28, 2025 20:50
im2nguyen pushed a commit that referenced this pull request Mar 6, 2025
…#96)

* Added function to collect code list items; added tests

* Moved isParagraph and isInlineCode to codeListItemCollector function
hashibot-web pushed a commit that referenced this pull request May 1, 2025
Add callout for private policy evaluation
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.

3 participants