Skip to content

fix: Update history rendering template from Liquid to Handlebars #182

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ismaeel-ch
Copy link

Description:
Updated the history rendering template in the Semantic Kernel GitHub prompt section to use Handlebars syntax ({{#each}}) instead of Liquid ({% for %}). This change ensures compatibility with Handlebars, which does not support the Liquid templating format.

  • Replaced {% for item in history %} with {{#each history}} for iteration.
  • Updated property references from item.role and item.content to this.role and this.content. This fix resolves template parsing errors in environments using Handlebars.

Testing:

Verified proper rendering of history data after template adjustment. Ensured no syntax errors in Handlebars template rendering.

Description:
Updated the history rendering template in the Semantic Kernel GitHub prompt section to use Handlebars syntax ({{#each}}) instead of Liquid ({% for %}). This change ensures compatibility with Handlebars, which does not support the Liquid templating format.

Replaced {% for item in history %} with {{#each history}} for iteration.
Updated property references from item.role and item.content to this.role and this.content.
This fix resolves template parsing errors in environments using Handlebars.

Testing:

Verified proper rendering of history data after template adjustment.
Ensured no syntax errors in Handlebars template rendering.
Copy link
Contributor

Learn Build status updates of commit 62d39de:

✅ Validation status: passed

File Status Preview URL Details
semantic-kernel/concepts/prompts/handlebars-prompt-templates.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

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.

1 participant