Skip to content

Conversation

@harshach
Copy link
Collaborator

@harshach harshach commented Dec 28, 2025

Describe your changes:

Fixes

I worked on ... because ...

Type of change:

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Summary by Gitar

  • New backend entity:
    • LearningResource with JSON schema, repository, REST API endpoints, and MySQL/PostgreSQL migrations for managing in-product educational content
  • Frontend components:
    • LearningIcon with badge count, LearningDrawer for contextual resource listing, and ResourcePlayerModal supporting Article/Video/Storylane formats
  • Admin interface:
    • Full CRUD page at /settings/learning-resources with table view, form validation, and markdown editor for managing learning resources
  • Page integrations:
    • Added learning icon to 10+ pages including Glossary, Domain, Data Products, Data Quality, Observability, Policies, and Roles
  • Sample content:
    • 25+ seed resources with merge strategy covering tutorials, demos, and integration guides

This will update automatically on new commits.


@github-actions
Copy link
Contributor

github-actions bot commented Dec 28, 2025

Jest test Coverage

UI tests summary

Lines Statements Branches Functions
Coverage: 64%
64.7% (51697/79899) 42.46% (25249/59472) 45.88% (7977/17385)

…ing sanitization

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@gitar-bot
Copy link

gitar-bot bot commented Dec 28, 2025

The Test Report failures from jobs 59043565494, 59043568500, 59043569547, and 59043569614 are from commit fdb78adf which has been superseded by the merge to main in commit 5a48c1f4.

Current CI status for the latest commit:

  • ✅ Several workflows passed (Maven MySQL/Postgres Tests, TypeScript Generation, Validate JSON/YAML, etc.)
  • ⏳ Multiple workflows still in progress (py-tests, Playwright E2E, Integration Tests, etc.)
  • ❌ One failure: .github/workflows/playwright-sso-tests.yml (run 20557456283)

The old Test Report failures are no longer relevant as they're from a superseded commit.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 2, 2026

TypeScript types have been updated based on the JSON schema changes in the PR

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 2, 2026

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 2, 2026

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

Labels

backend safe to test Add this label to run secure Github workflows on PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants