Skip to content

Support typed injection in Angular #7651

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

Closed
wants to merge 2 commits into from
Closed

Conversation

shylasummers
Copy link
Contributor

@shylasummers shylasummers commented Mar 19, 2025

This PR addresses the issue reported by @bradkovach and uses the solution suggested in #7484

@github-actions github-actions bot added the msal-angular Related to @azure/msal-angular package label Mar 19, 2025
@shylasummers shylasummers marked this pull request as ready for review March 19, 2025 21:05
@shylasummers shylasummers changed the title Adjust injection token constants to support typed injection in Angular Support typed injection in Angular Mar 19, 2025
@bradkovach
Copy link

Please merge my PR #7484 instead of taking my implementation and submitting a PR for my work under your own name.

@shylasummers
Copy link
Contributor Author

Hi @bradkovach, we made a separate PR because we needed to make a different change file and because we're not merging this PR into the dev branch.

@bradkovach
Copy link

bradkovach commented Mar 19, 2025

Please change the target branch on #7484 and do this the right way.

Source branch changed, so a new PR was created. Please use #7653

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Attention 👋 Awaiting response from the MSAL.js team label Apr 7, 2025
@shylasummers
Copy link
Contributor Author

Closing in favor of #7745, which preserves commit history

@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs: Attention 👋 Awaiting response from the MSAL.js team label May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
msal-angular Related to @azure/msal-angular package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants