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

feat: replace any typescript file by configuration #616

Merged
merged 1 commit into from
Mar 24, 2021

Conversation

dhhyi
Copy link
Collaborator

@dhhyi dhhyi commented Mar 12, 2021

PR Type

[x] Feature

What Is the Current Behavior?

Environment specific overrides (#537) can only override HTML templates and component styles

What Is the New Behavior?

Any typescript file can be overridden by having the extension ".ts".

File overrides can be active for multiple configurations. For example custom.service.brand1.brand2.ts will activate for ng build -c brand1 and also for ng build -c brand2.

Does this PR Introduce a Breaking Change?

[ ] Yes
[ ] No

Other Information

This is useful for projects for overriding functionality without introducing changes in ISH sources.

@dhhyi dhhyi added the enhancement Enhancement to an existing feature label Mar 12, 2021
@dhhyi dhhyi self-assigned this Mar 12, 2021
@dhhyi dhhyi force-pushed the feat/replace-typescript-files-by-config branch from 5f2d22c to 2232870 Compare March 17, 2021 08:49
@dhhyi dhhyi force-pushed the feat/replace-typescript-files-by-config branch from 9b92a83 to b84211a Compare March 17, 2021 13:23
@dhhyi dhhyi removed their assignment Mar 17, 2021
@dhhyi dhhyi marked this pull request as ready for review March 17, 2021 14:18
@dhhyi dhhyi requested review from MaxKless and shauke March 17, 2021 14:18
@shauke shauke self-assigned this Mar 18, 2021
@shauke shauke added this to the 0.29 milestone Mar 18, 2021
Copy link
Collaborator

@shauke shauke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice feature, but some additional documentation would be nice too. ;)

@shauke shauke removed the request for review from MaxKless March 19, 2021 12:45
@dhhyi
Copy link
Collaborator Author

dhhyi commented Mar 19, 2021

Nice feature, but some additional documentation would be nice too. ;)

Okay 😄

@dhhyi dhhyi marked this pull request as draft March 19, 2021 20:45
@dhhyi dhhyi force-pushed the feat/replace-typescript-files-by-config branch from 02a1c09 to fb0d45b Compare March 19, 2021 21:58
@dhhyi dhhyi force-pushed the feat/replace-typescript-files-by-config branch from fb0d45b to 2cf4438 Compare March 19, 2021 23:20
@dhhyi
Copy link
Collaborator Author

dhhyi commented Mar 20, 2021

I took the opportunity to also sneak in an optimization and a test. 👍

@dhhyi dhhyi marked this pull request as ready for review March 20, 2021 10:36
@dhhyi dhhyi requested a review from shauke March 20, 2021 10:36
@shauke shauke merged commit 7b39473 into develop Mar 24, 2021
@shauke shauke deleted the feat/replace-typescript-files-by-config branch March 24, 2021 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants