Back to pull request #31521

regression: Convert emoji shortname to unicode on notification emails #16209

Triggered via pull request January 23, 2024 19:20

gabriellsh

opened #31521

fix/sendNotification

Status Success

Total duration 40m 5s

Artifacts 3

ci.yml

on: pull_request

⚙️ Variables Setup

🚀 Notify external services - draft

📦 Build Packages

44s

Matrix: 🔎 Code Check / code-check

📦 Meteor Build - coverage

13m 22s

🔨 Test Unit / Unit Tests

1m 22s

Matrix: 🚢 Build Docker Images for Testing

🚢 Build Docker Image (preview)

Matrix: 🔨 Test API (EE) / test

Matrix: 🔨 Test API (CE) / test

Matrix: 🔨 Test UI (EE) / test

Matrix: 🔨 Test UI (CE) / test

✅ Tests Done

📦 Meteor Build - official

Matrix: 🚢 Build Docker Images for Production

🚀 Publish build assets

Matrix: 🚀 Publish Docker Image (main)

Matrix: 🚀 Publish Docker Image (services)

🚀 Notify external services

trigger-dependent-workflows

Annotations

23 warnings

🔎 Code Check / Code Lint: packages/livechat/src/components/Composer/index.tsx#L29

Prefer using an optional chain expression instead, as it's more concise and easier to read

🔎 Code Check / Code Lint: packages/livechat/src/components/Composer/index.tsx#L244

Forbidden non-null assertion

🔎 Code Check / Code Lint: packages/livechat/src/components/Composer/index.tsx#L252

Forbidden non-null assertion

🔎 Code Check / Code Lint: apps/meteor/app/api/server/v1/settings.ts#L109

Prefer using an optional chain expression instead, as it's more concise and easier to read

🔎 Code Check / Code Lint: apps/meteor/app/apps/server/bridges/listeners.js#L10

Arrow function has a complexity of 40. Maximum allowed is 31

🔎 Code Check / Code Lint: apps/meteor/app/apps/server/bridges/persistence.ts#L92

Prefer using an optional chain expression instead, as it's more concise and easier to read

🔎 Code Check / Code Lint: apps/meteor/app/apps/server/bridges/rooms.ts#L82

Prefer using an optional chain expression instead, as it's more concise and easier to read

🔎 Code Check / Code Lint: apps/meteor/app/apps/server/bridges/rooms.ts#L94

Prefer using an optional chain expression instead, as it's more concise and easier to read

🔎 Code Check / Code Lint: apps/meteor/app/apps/server/bridges/rooms.ts#L201

Forbidden non-null assertion

🔎 Code Check / Code Lint: apps/meteor/app/apps/server/bridges/rooms.ts#L202

Forbidden non-null assertion

🔨 Test UI (CE) / MongoDB 4.4 (3/4)