Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correctly send the created PR slacks #2566

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

mokimo
Copy link
Contributor

@mokimo mokimo commented Jul 12, 2024

Correctly pass on the milo-stage slack webhook url to the workflow
Related: #2474
Resolves: MWPW-152624

Test URLs:

Copy link
Contributor

aem-code-sync bot commented Jul 12, 2024

Page Scores Audits Google
/?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

Copy link
Contributor

This pull request is not passing all required checks. Please see this discussion for information on how to get all checks passing. Inconsistent checks can be manually retried. If a test absolutely can not pass for a good reason, please add a comment with an explanation to the PR.

@mokimo mokimo force-pushed the stage-pr-created-to-slack-stage branch from 0a35699 to 4d12337 Compare July 15, 2024 15:04
Copy link

codecov bot commented Jul 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.87%. Comparing base (a856073) to head (4d12337).

Additional details and impacted files
@@            Coverage Diff             @@
##            stage    #2566      +/-   ##
==========================================
- Coverage   95.87%   95.87%   -0.01%     
==========================================
  Files         176      176              
  Lines       46133    46133              
==========================================
- Hits        44229    44228       -1     
- Misses       1904     1905       +1     

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

Copy link
Contributor

Reminder to set the Ready for Stage label - to queue this to get merged to stage & production.

@mokimo mokimo merged commit fc03376 into adobecom:stage Jul 16, 2024
11 checks passed
rohitsahu pushed a commit to rohitsahu/milo that referenced this pull request Jul 16, 2024
* stage:
  Mwpw-142267:  Merch What's Included and Merch Mnemonic List (TwP) (adobecom#2554)
  MWPW-149124 Improve Focus Page for Performance Improvement Tiger Team  (adobecom#2391)
  Correctly send the created PR slacks (adobecom#2566)
  MWPW-153167: caas-config change to enable hiding date detail information (adobecom#2553)
  MWPW-152016 - Localization target preview for transcreation (adobecom#2564)
  Relax CORS restrictions for module imports (adobecom#2549)
  MWPW-153600 [PEP] loader bar on PEP prompt is seen loaded Left to right in RTL locale (adobecom#2548)
  MWPW-146962 [milo] Text link behaving like button in FAQ section (adobecom#2530)
  MWPW-152280 MEP: Only preload fragments that are in the 1st section (adobecom#2525)
  MWPW-152697 Fix Marketo mobile horizontal scroll (adobecom#2514)
  MWPW-151513: Search results vanished when user clicks on Marquee CTA:Start free trail (adobecom#2406)
  MWPW-154013 PEP prompt redirection is broken in stage after the PEP dismissal PR merge (adobecom#2547)
  MWPW-153962: Introduce maslibs query parameter (adobecom#2544)
  Central georouting support (adobecom#2531)
  [MWPW-152278] Avoid empty CSS requests (adobecom#2524)
  MWPW-152918 Fix Marketo button font (adobecom#2513)

# Conflicts:
#	libs/deps/merch-card.js
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