Skip to content

Remove custom lwc from Migrate, Assess and Reports#282

Merged
sf-pranav-varshney merged 9 commits intoprerelease/alphafrom
remove_custom_lwc
Jun 11, 2025
Merged

Remove custom lwc from Migrate, Assess and Reports#282
sf-pranav-varshney merged 9 commits intoprerelease/alphafrom
remove_custom_lwc

Conversation

@rajender-kumar-salesforce
Copy link
Copy Markdown
Collaborator

@rajender-kumar-salesforce rajender-kumar-salesforce commented Jun 9, 2025

Commented out the code of LWC migration, LWC assessment and LWC report generation because of migration MVP.

What issues does this PR fix or reference?](https://gus.lightning.force.com/lightning/r/ADM_Work__c/a07EE00002E9FvJYAV/view)

@rajender-kumar-salesforce rajender-kumar-salesforce requested a review from a team as a code owner June 9, 2025 04:07
@salesforce-cla
Copy link
Copy Markdown

salesforce-cla bot commented Jun 9, 2025

Thanks for the contribution! It looks like @vsushmita is an internal user so signing the CLA is not required. However, we need to confirm this.

@rajender-kumar-salesforce rajender-kumar-salesforce changed the base branch from main to prerelease/alpha June 9, 2025 04:07
@sf-pranav-varshney
Copy link
Copy Markdown

@rajender-kumar-salesforce we also need to update the ReadMe for LWC.

const projectDirectory = OmnistudioRelatedObjectMigrationFacade.intializeProject();
conn.setApiVersion(apiVersion);
// @ts-expect-error - LWC functionality temporarily disabled
const lwcparser = new LwcMigration(projectDirectory, namespace, this.org);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Instead of adding a lint annotation.

I would recommend, we can simply comment out the LWC processing — it's unnecessary and also slows down the assessment of omnistudio components.

// Initialize migration tools based on the relatedObjects parameter
const apexMigrator = this.createApexClassMigrationTool(projectDirectory, targetApexNamespace);
// @ts-expect-error - LWC functionality temporarily disabled
const lwcMigrator = this.createLWCComponentMigrationTool(this.namespace, projectDirectory);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same here.

@sf-pranav-varshney sf-pranav-varshney merged commit 50c5805 into prerelease/alpha Jun 11, 2025
8 checks passed
sf-kishore-kurri pushed a commit to sf-kishore-kurri/plugin-omnistudio-migration-tool that referenced this pull request Jun 24, 2025
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.

2 participants