Skip to content

Conversation

@nickolas-dimitrakas
Copy link
Contributor

@nickolas-dimitrakas nickolas-dimitrakas commented Dec 19, 2025

Background

  • The SDK already uses Scene Graph architecture internally, and Roku has standardized on Scene Graph for years. The legacy example adds unnecessary maintenance overhead and can confuse developers about the recommended approach.

What Has Changed

  • Removed example-legacy-sdk/ directory
  • Simplified deploy-and-test.sh to only support Scene Graph example
  • Updated README.md to remove legacy example references

Checklist

  • I have performed a self-review of my own code.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have tested this locally.

Additional Notes

Reference Issue (For employees only. Ignore if you are an outside contributor)

@nickolas-dimitrakas nickolas-dimitrakas self-assigned this Dec 19, 2025
@nickolas-dimitrakas nickolas-dimitrakas requested a review from a team as a code owner December 19, 2025 16:38
@nickolas-dimitrakas nickolas-dimitrakas changed the base branch from master to feat/media-content-time-spent-calculation December 19, 2025 16:47
@nickolas-dimitrakas nickolas-dimitrakas force-pushed the feat/media-content-time-spent-calculation branch from 9bfc9c2 to f8a1468 Compare December 23, 2025 00:01
Copy link
Collaborator

@jamesnrokt jamesnrokt left a comment

Choose a reason for hiding this comment

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

Approved pending changing the base branch and merging of #44

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants