Skip to content

Fix named injection#205

Merged
guyca merged 2 commits intomasterfrom
fixNamedInject
Mar 4, 2025
Merged

Fix named injection#205
guyca merged 2 commits intomasterfrom
fixNamedInject

Conversation

@guyca
Copy link
Collaborator

@guyca guyca commented Mar 4, 2025

When manually passing the injection token to the @Inject decorator, it was incorrectly overridden with the property name.

@Inject('FooToken') foo was transpiled by mistake as @Inject('foo') foo.

guyca added 2 commits March 2, 2025 18:43
The Babel plugin should not modify the custom name passed to @Inject and @LateInject
# Conflicts:
#	packages/react-obsidian/transformers/babel-plugin-obsidian/__snapshots__/index.test.ts.snap
@guyca guyca changed the title Fix named inject Fix named injection Mar 4, 2025
@guyca guyca merged commit 5a5ef2a into master Mar 4, 2025
2 checks passed
@guyca guyca deleted the fixNamedInject branch March 4, 2025 11:19
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.

1 participant