Skip to content

fix: replace variable accounName for cli login (#15694)#15697

Merged
wh-alice merged 1 commit intorelease/6.8from
helly/clilogin
Apr 13, 2026
Merged

fix: replace variable accounName for cli login (#15694)#15697
wh-alice merged 1 commit intorelease/6.8from
helly/clilogin

Conversation

@hellyzh
Copy link
Copy Markdown
Collaborator

@hellyzh hellyzh commented Apr 13, 2026

The ${accountName} placeholders are now replaced with the display name ("M365" or "Azure") before passing the template to MSAL's successTemplate/errorTemplate

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.94%. Comparing base (11924e7) to head (0bd51e1).
⚠️ Report is 2 commits behind head on release/6.8.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff               @@
##           release/6.8   #15697      +/-   ##
===============================================
- Coverage        92.95%   92.94%   -0.01%     
===============================================
  Files              623      623              
  Lines            38443    38443              
  Branches          6957     6978      +21     
===============================================
- Hits             35735    35732       -3     
  Misses            1979     1979              
- Partials           729      732       +3     

see 2 files with indirect coverage changes

🚀 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.

@wh-alice wh-alice merged commit df8cdda into release/6.8 Apr 13, 2026
67 of 91 checks passed
@wh-alice wh-alice deleted the helly/clilogin branch April 13, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants