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.

Sign up for GitHub

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

Jump to bottom

fixed env config renaming #7823

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
+27 −36
Open

fixed env config renaming #7823

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
dd65bec
fixed env config renaming
NitsanCohen770 Aug 22, 2023
4a585e8
fix if check and env with version
NitsanCohen770 Aug 23, 2023
bb9784c
fix replace env with version on bitmap
NitsanCohen770 Aug 23, 2023
3cc1424
fix rename new component issue
NitsanCohen770 Aug 23, 2023
eddaae8
Merge branch 'master' into fix-bitmap-env-rename
NitsanCohen770 Aug 23, 2023
4c52a07
remove redundant getBitmapEntry
NitsanCohen770 Aug 23, 2023
1017585
Merge branch 'master' into fix-bitmap-env-rename
NitsanCohen770 Aug 23, 2023
3b76652
replace the method with new workspace method
NitsanCohen770 Aug 23, 2023
610fcc4
remove renameAspectInConfig as it is not used now
NitsanCohen770 Aug 23, 2023
0dbb915
remove import
NitsanCohen770 Aug 23, 2023
24fdf83
fix rename new components
NitsanCohen770 Aug 23, 2023
b0b110f
Merge branch 'master' into fix-bitmap-env-rename
NitsanCohen770 Aug 23, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
5 changes: 3 additions & 2 deletions scopes/workspace/workspace/bit-map.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -192,14 +192,15 @@ export class BitMap {
const config = componentMap.config;
if (!config) return;
Object.keys(config).forEach((aspectId) => {
if (aspectId === sourceId.toString()) {
const fullSourceId = this.getBitmapEntry(sourceId).id.toStringWithoutVersion();
if (aspectId === fullSourceId) {
config[targetId.toString()] = config[aspectId];
delete config[aspectId];
this.markAsChanged();
}
if (aspectId === EnvsAspect.id) {
const envConfig = config[aspectId];
if (envConfig !== REMOVE_EXTENSION_SPECIAL_SIGN && envConfig.env === sourceId.toString()) {
if (envConfig !== REMOVE_EXTENSION_SPECIAL_SIGN && envConfig.env !== sourceId.toString()) {
Copy link
Member

Choose a reason for hiding this comment

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

Why this need be!== and not ===
With!== it will replace any env for any component to the new one

Copy link
Contributor Author

Choose a reason for hiding this comment

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

because it works? 🤣
envConfig.env is the env prop in the config for the currently iterated component in the bitmap.
the sourceId (should change it to the fullSoruceId) is the env we want to change.
So you are right and I have to fix it now 😨 (I was testing it on one component)

Fixed on next commit. Also fixed a case where the source env is specified with a version and it wouldn't detect it.

envConfig.env = targetId.toString();
this.markAsChanged();
}
Expand Down