Skip to content

Add new asset process to README#7012

Merged
trmartin4 merged 1 commit intomainfrom
platform/add-asset-process
Feb 17, 2026
Merged

Add new asset process to README#7012
trmartin4 merged 1 commit intomainfrom
platform/add-asset-process

Conversation

@trmartin4
Copy link
Member

@trmartin4 trmartin4 commented Feb 16, 2026

📔 Objective

Updating the process for managing email assets now that we support using git for uploading assets.

⏰ Reminders before review

  • Contributor guidelines followed
  • All formatters and local linters executed and passed
  • Written new unit and / or integration tests where applicable
  • Protected functional changes with optionality (feature flags)
  • Used internationalization (i18n) for all UI strings
  • CI builds passed
  • Communicated to DevOps any deployment requirements
  • Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

  • 👍 (:+1:) or similar for great changes
  • 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
  • ❓ (:question:) for questions
  • 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
  • 🎨 (:art:) for suggestions / improvements
  • ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
  • 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
  • ⛏ (:pick:) for minor or nitpick changes

@trmartin4 trmartin4 changed the title Added new asset dopcs to README. Adding new asset process to README Feb 16, 2026
@trmartin4 trmartin4 marked this pull request as ready for review February 16, 2026 20:02
@trmartin4 trmartin4 added the ai-review Request a Claude code review label Feb 16, 2026
@trmartin4 trmartin4 changed the title Adding new asset process to README Add new asset process to README Feb 16, 2026
@sonarqubecloud
Copy link

@trmartin4 trmartin4 requested a review from watsondm February 16, 2026 20:11
@github-actions
Copy link
Contributor

Logo
Checkmarx One – Scan Summary & Details86f36d91-ec13-4c5e-a1ea-2910811d6806

Great job! No new security vulnerabilities introduced in this pull request

@codecov
Copy link

codecov bot commented Feb 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.46%. Comparing base (b03f8f8) to head (25b00c3).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7012   +/-   ##
=======================================
  Coverage   56.46%   56.46%           
=======================================
  Files        1996     1996           
  Lines       88039    88039           
  Branches     7845     7845           
=======================================
  Hits        49715    49715           
  Misses      36487    36487           
  Partials     1837     1837           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@trmartin4 trmartin4 merged commit 3ae0402 into main Feb 17, 2026
47 checks passed
@trmartin4 trmartin4 deleted the platform/add-asset-process branch February 17, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants