Skip to content

fix: Read ASR and DTMF if not included#20

Merged
Ruari-Phipps merged 2 commits intomainfrom
ruari/fix/read_asr_dtmf_if_not_none
Mar 16, 2026
Merged

fix: Read ASR and DTMF if not included#20
Ruari-Phipps merged 2 commits intomainfrom
ruari/fix/read_asr_dtmf_if_not_none

Conversation

@Ruari-Phipps
Copy link
Copy Markdown
Collaborator

@Ruari-Phipps Ruari-Phipps commented Mar 16, 2026

Summary

If ASR and DTMF not included, still read them as defaults

Motivation

Would error if created without these fields

Changes

  • Filter on None not empty dicts

Test strategy

  • Added/updated unit tests
  • Manual CLI testing (poly <command>)
  • Tested against a live Agent Studio project
  • N/A (docs, config, or trivial change)

Checklist

  • ruff check . and ruff format --check . pass
  • pytest passes
  • No breaking changes to the poly CLI interface (or migration path documented)
  • Commit messages follow conventional commits

Screenshots / Logs

@Ruari-Phipps Ruari-Phipps merged commit bc5add7 into main Mar 16, 2026
2 checks passed
@Ruari-Phipps Ruari-Phipps deleted the ruari/fix/read_asr_dtmf_if_not_none branch March 16, 2026 16:26
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