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

fix(modal): provide modal config service in root and export (backport to 16.x) #1559

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Sep 19, 2024

This is a backport of 29eed4c (#1556) to 16.x.

What is the current behavior?

The following error occurs when using VmwNgxModalService:

ERROR NullInjectorError: R3InjectorError(Environment Injector)[ClrModalConfigurationService -> ClrModalConfigurationService]: 
  NullInjectorError: No provider for ClrModalConfigurationService!
    at NullInjector.get (core.mjs:1635:21)
    at R3Injector.get (core.mjs:3017:27)
    at R3Injector.get (core.mjs:3017:27)
    at ChainedInjector.get (core.mjs:5288:32)
    at lookupTokenUsingModuleInjector (core.mjs:5631:31)
    at getOrCreateInjectable (core.mjs:5677:10)
    at ɵɵdirectiveInject (core.mjs:11586:17)
    at NodeInjectorFactory.ClrModal_Factory [as factory] (clr-angular.mjs:32885:177)
    at getNodeInjectable (core.mjs:5871:38)
    at instantiateAllDirectives (core.mjs:12403:23)

Issue Number: CDE-2293

What is the new behavior?

No error occurs when using VmwNgxModalService.

## What is the current behavior?

The following error occurs when using `VmwNgxModalService`:

```
ERROR NullInjectorError: R3InjectorError(Environment Injector)[ClrModalConfigurationService -> ClrModalConfigurationService]:
  NullInjectorError: No provider for ClrModalConfigurationService!
    at NullInjector.get (core.mjs:1635:21)
    at R3Injector.get (core.mjs:3017:27)
    at R3Injector.get (core.mjs:3017:27)
    at ChainedInjector.get (core.mjs:5288:32)
    at lookupTokenUsingModuleInjector (core.mjs:5631:31)
    at getOrCreateInjectable (core.mjs:5677:10)
    at ɵɵdirectiveInject (core.mjs:11586:17)
    at NodeInjectorFactory.ClrModal_Factory [as factory] (clr-angular.mjs:32885:177)
    at getNodeInjectable (core.mjs:5871:38)
    at instantiateAllDirectives (core.mjs:12403:23)
```

Issue Number: CDE-2293

## What is the new behavior?

No error occurs when using `VmwNgxModalService`.

(cherry picked from commit 29eed4c)
Copy link
Contributor Author

github-actions bot commented Sep 19, 2024

👋 @kevinbuhmann,

  • 🙏 The Clarity team thanks you for opening a pull request
  • 🎉 The build for this PR has succeeded
  • 🔍 The PR is now ready for review
  • 🍿 In the meantime, view a preview of this PR
  • 🖐 You can always follow up here. If you're a VMware employee, you can also reach us on our internal Clarity Support space

Thank you,

🤖 Clarity Release Bot

@kevinbuhmann kevinbuhmann merged commit 8d611a8 into 16.x Sep 19, 2024
8 checks passed
@kevinbuhmann kevinbuhmann deleted the backport-1556-to-16.x branch September 19, 2024 18:34
Copy link
Contributor Author

github-actions bot commented Oct 2, 2024

🎉 This PR is included in version 16.5.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link
Contributor Author

Hi there 👋, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed PRs after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary.

@github-actions github-actions bot locked and limited conversation to collaborators Oct 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant