Skip to content

Feature/podcast player#235

Open
SpkArtZen wants to merge 10 commits intoopenhome-dev:devfrom
SpkArtZen:feature/podcast_player
Open

Feature/podcast player#235
SpkArtZen wants to merge 10 commits intoopenhome-dev:devfrom
SpkArtZen:feature/podcast_player

Conversation

@SpkArtZen
Copy link
Copy Markdown

What does this Ability do?

Suggested Trigger Words

Type

  • New community Ability
  • Improvement to existing Ability
  • Bug fix
  • Documentation update

External APIs

  • No external APIs
  • Uses external API(s):

Testing

  • Tested in OpenHome Live Editor
  • All exit paths tested (said "stop", "exit", etc.)
  • Error scenarios tested (API down, bad input, etc.)

Checklist

  • Files are in community/my-ability-name/
  • main.py follows SDK pattern (extends MatchingCapability, has register_capability + call)
  • README.md included with description, suggested triggers, and setup
  • resume_normal_flow() called on every exit path
  • No print() — using editor_logging_handler
  • No hardcoded API keys — using placeholders
  • No blocked imports (redis, user_config)
  • No asyncio.sleep() or asyncio.create_task() — using session_tasks
  • Error handling on all external calls
  • Tested in OpenHome Live Editor

Anything else?

@SpkArtZen SpkArtZen requested a review from a team as a code owner March 30, 2026 12:14
@github-actions
Copy link
Copy Markdown
Contributor

🔀 Branch Merge Check

PR direction: feature/podcast_playerdev

Passedfeature/podcast_playerdev is a valid merge direction

@github-actions
Copy link
Copy Markdown
Contributor

✅ Ability Validation Passed

📋 Validating: community/podcast-player
  ✅ All checks passed!

@github-actions
Copy link
Copy Markdown
Contributor

✅ Community PR Path Check — Passed

All changed files are inside the community/ folder. Looks good!

@github-actions github-actions bot added the community-ability Community-contributed ability label Mar 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Lint Results

__init__.py — Empty as expected

Files linted: community/podcast-player/main.py

✅ Flake8 — Passed

✅ All checks passed!

Copy link
Copy Markdown
Contributor

@uzair401 uzair401 left a comment

Choose a reason for hiding this comment

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

Hi @SpkArtZen, it looks like a new PR was opened without addressing the previously requested changes. Please update your implementation based on the earlier feedback and include a working demo recording as well. Once done, we’ll proceed with the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-ability Community-contributed ability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants