Skip to content

Comments

Add outlook daily brief#102

Open
RyanBhandal wants to merge 13 commits intoopenhome-dev:devfrom
RyanBhandal:outlook-daily-brief
Open

Add outlook daily brief#102
RyanBhandal wants to merge 13 commits intoopenhome-dev:devfrom
RyanBhandal:outlook-daily-brief

Conversation

@RyanBhandal
Copy link
Contributor

@RyanBhandal RyanBhandal commented Feb 20, 2026

What does this Ability do?

Gives you a daily brief of what you have coming up in the day using both the weather and Outlook!

Suggested Trigger Words

  • Daily Brief

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, connection_manager, 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?

(https://www.loom.com/share/139448587ddb41c6ac08e4e1830df3e8)

@RyanBhandal RyanBhandal requested a review from a team as a code owner February 20, 2026 13:06
@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

🔀 Branch Merge Check

PR direction: outlook-daily-briefdev

Passedoutlook-daily-briefdev is a valid merge direction

@github-actions github-actions bot added the first-contribution First-time contributor label Feb 20, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

✅ Ability Validation Passed

📋 Validating: community/outlook-daily-brief
  ✅ All checks passed!

@github-actions github-actions bot added the community-ability Community-contributed ability label Feb 20, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

🔍 Lint Results

__init__.py — Empty as expected

Files linted: community/outlook-daily-brief/main.py community/outlook-daily-brief/refresh_token/get_refresh_token.py

✅ Flake8 — Passed

✅ All checks passed!

@Rizwan-095
Copy link
Contributor

Kindly look at the validate_ability.py file in root directory of this repo again there are few changes so build your ability according to that.
Also look at these docs.
https://docs.openhome.com/how_to_build_an_ability

@RyanBhandal RyanBhandal requested a review from a team as a code owner February 20, 2026 19:39
@RyanBhandal RyanBhandal force-pushed the outlook-daily-brief branch 2 times, most recently from 93dc0c9 to 6e10a7f Compare February 20, 2026 19:41
@RyanBhandal
Copy link
Contributor Author

Kindly look at the validate_ability.py file in root directory of this repo again there are few changes so build your ability according to that. Also look at these docs. https://docs.openhome.com/how_to_build_an_ability

Are you sure? They seem to pass for me?

@Rizwan-095
Copy link
Contributor

#102 (comment)
Look at this validation still failing

@RyanBhandal
Copy link
Contributor Author

Code has been updated - thanks

@RyanBhandal
Copy link
Contributor Author

@Rizwan-095 could you please take a look at this ability again please

@codermohsin
Copy link
Contributor

Your ability is under review.

Our team will get back to you and give feedback.

Thanks

@github-actions
Copy link
Contributor

github-actions bot commented Feb 24, 2026

✅ Community PR Path Check — Passed

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

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

Labels

community-ability Community-contributed ability first-contribution First-time contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants