Skip to content

Create MicrosoftTeamsListMessages action#42475

Merged
inbalapt1 merged 39 commits intomasterfrom
MicrosoftTeamsListMessages
Jan 28, 2026
Merged

Create MicrosoftTeamsListMessages action#42475
inbalapt1 merged 39 commits intomasterfrom
MicrosoftTeamsListMessages

Conversation

@inbalapt1
Copy link
Contributor

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

link to the issue

Description

Must have

  • Tests
  • Documentation

@inbalapt1 inbalapt1 requested a review from dantavori as a code owner December 30, 2025 13:52
@inbalapt1 inbalapt1 self-assigned this Dec 30, 2025
@github-actions
Copy link

github-actions bot commented Dec 30, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/MicrosoftTeams/Integrations/MicrosoftTeams
   MicrosoftTeams.py173439177%309, 315, 378, 380, 420–421, 444, 453, 455, 483–484, 486–489, 491–494, 498, 516, 520, 528–529, 746–747, 780, 803, 896–899, 901–902, 917–918, 927–928, 966, 977, 984–986, 989, 998–999, 1020–1021, 1056–1058, 1060–1061, 1072, 1085–1088, 1091–1093, 1096–1098, 1102, 1111–1112, 1116–1117, 1142, 1156–1163, 1165–1169, 1171–1173, 1175–1177, 1180–1183, 1185, 1187–1204, 1206, 1208–1209, 1212–1215, 1217, 1219–1220, 1222–1225, 1227–1228, 1230, 1235, 1237–1241, 1243–1245, 1347–1350, 1358–1361, 1363–1365, 1367–1370, 1372, 1374, 1425, 1427, 1554–1555, 1567, 1639, 1712, 1714, 1717–1719, 1721–1729, 1731–1732, 1734, 1737, 1756, 1761, 1763, 1943–1944, 1950, 1963, 2003, 2182–2183, 2267–2272, 2291–2303, 2404, 2425–2426, 2492, 2501, 2531–2534, 2557–2559, 2572, 2592, 2595, 2598, 2606, 2609–2611, 2617, 2641–2642, 2645, 2694–2696, 2698, 2733–2740, 2742–2748, 2753–2754, 2756–2765, 2769–2772, 2774, 2800–2801, 2814–2816, 2846, 2852–2854, 2861, 2872–2873, 2876, 2880, 2918–2920, 2967–2968, 2971, 3042, 3046, 3058, 3075–3077, 3097, 3144–3145, 3159, 3165–3167, 3187–3189, 3199, 3201–3208, 3210–3212, 3216–3219, 3245, 3250, 3305–3307, 3309–3310, 3312, 3314–3315, 3317–3318, 3320–3322, 3324, 3326–3330, 3332–3335, 3337–3339, 3341, 3343, 3345–3355, 3357–3363, 3442, 3459, 3504–3507, 3510–3511, 3516, 3526–3527, 3538–3539, 3541, 3550, 3576, 3650, 3675, 3679, 3690, 3693, 3710–3711
TOTAL173439177% 

Tests Skipped Failures Errors Time
146 0 💤 0 ❌ 0 🔥 4.534s ⏱️

@content-bot
Copy link
Contributor

This PR was automatically updated by a GitHub Action

  • MicrosoftTeams pack version was bumped to 1.5.51.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@inbalapt1 inbalapt1 added ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. labels Jan 12, 2026
@content-bot content-bot removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Jan 12, 2026
@content-bot
Copy link
Contributor

🤖 Content AI Reviewer: Analysis started. Please wait for results...

content-bot

This comment was marked as outdated.

@inbalapt1 inbalapt1 added the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Jan 14, 2026
@content-bot content-bot removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Jan 14, 2026
@content-bot
Copy link
Contributor

🤖 Content AI Reviewer: Analysis started. Please wait for results...

@inbalapt1 inbalapt1 added the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Jan 14, 2026
@content-bot content-bot removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Jan 14, 2026
Copy link
Contributor

@content-bot content-bot left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution to the Microsoft Teams pack! I have a few comments regarding the new command and documentation.

Since this introduces new functionality, please update the version to a minor bump (1.6.0). Additionally, I noticed some opportunities to tighten up exception handling, verify permissions for the Client Credentials flow, and align the argument names in the README with the YAML configuration.

Great work so far!

@dantavori, @omerKarkKatz please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

@demisto demisto deleted a comment from content-bot Jan 15, 2026
@demisto demisto deleted a comment from content-bot Jan 15, 2026
@demisto demisto deleted a comment from content-bot Jan 15, 2026
@demisto demisto deleted a comment from content-bot Jan 15, 2026
@demisto demisto deleted a comment from content-bot Jan 15, 2026
@demisto demisto deleted a comment from content-bot Jan 15, 2026
@demisto demisto deleted a comment from content-bot Jan 15, 2026
@itssapir itssapir self-requested a review January 20, 2026 13:01
Copy link
Contributor

@itssapir itssapir left a comment

Choose a reason for hiding this comment

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

Looks great! see my comments

@content-bot
Copy link
Contributor

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

@inbalapt1 inbalapt1 merged commit 3070e70 into master Jan 28, 2026
21 checks passed
@inbalapt1 inbalapt1 deleted the MicrosoftTeamsListMessages branch January 28, 2026 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants