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

Unable to handle email templates #546

Closed
1 task
ciureanub opened this issue Mar 30, 2023 · 3 comments · Fixed by #547
Closed
1 task

Unable to handle email templates #546

ciureanub opened this issue Mar 30, 2023 · 3 comments · Fixed by #547
Assignees
Labels
bug Something isn't working

Comments

@ciureanub
Copy link

ciureanub commented Mar 30, 2023

Issue verification check:

  • is the current repository fully deployable at the commit SHA provided with the 'from' parameter of the command?
  • We're conducting delta-deployments for our project while parallelly retrieving prod metadata into branches to bring them closer to deployability-state.

What is the problem?

We've noticed an incorrect sfdx-git-delta plugin behavior when it comes to email templates.

Using the plugin to get delta from PRs resulted in missing .email files (can be 0in size), which are required to deploy .email-meta.xml files. In other words any email template consists of two files:
email_template_developer_name.email
email_template_developer_name.email-meta.xml
The plugin generated delta only with .email-meta.xml files, skipping .email complementary files --> It's not possible to deploy only .email-meta.xml files. .email files have to be present to make it a deployable metadata delta.

It turned out that the last known working plugin version is 5.7.1. It could generate a delta of the pull request without issues (including .email).
Many higher plugin versions (we checked above 7), including the newest one generated delta without .email files.

It seemed as a plugin fault at the very first sight however creating a new email template in a clean scratch org, retrieving it, committing and generating a delta resulted in correct outcome (both .email and .email-meta.xml files were present). I'm not sure if it's a bug in the plugin that has been hidden for so many releases.

We're in the middle of retrieving all email templates and email folders from production . Once that's done we should test the scenario once again and see if the plugin generates delta correctly.


What is the parameter and the value you used with it?

What is the expected result?

What is the actual result?

No .email file is copied to delta folder

Steps to reproduce


Execution context


Operating System: Windows
node version: node v19.0.1
git version: git version 2.39.2.windows.1
sfdx version: sfdx-cli/7.193.2 win32-x64 node-v18.14.2
sgd plugin version: sfdx-git-delta 5.13.3

More information (optional)


GRPCL_AAL_EN
GRPCL_AAL_EN_333_program_step1.email-meta.xml
GRPCL_AAL_EN_333_program_step2.email-meta.xml
GRPCL_AAL_EN_333_program_step3.email-meta.xml
GRPCL_AAL_EN_Birthday.email-meta.xml
GRPCL_AAL_EN_Prospect_program_step1.email-meta.xml
GRPCL_AAL_EN_Prospect_program_step2.email-meta.xml
GRPCL_AAL_EN_Prospect_program_step3.email-meta.xml
GRPCL_AAL_EN_Share_a_product.email-meta.xml
GRPCL_AAL_EN_Win_back_program_step1.email-meta.xml
GRPCL_AAL_EN_Win_back_program_step2.email-meta.xml

@ciureanub ciureanub added the bug Something isn't working label Mar 30, 2023
@scolladon
Copy link
Owner

scolladon commented Mar 30, 2023

Hi @ciureanub !

Thanks for raising this issue and thanks for contributing in making this project better!

I'm able to reproduce locally when .email file is empty, they are not copied alongside the .email-meta.xml related file...

I'll ship a fix soon
Stay tuned !

@ciureanub
Copy link
Author

@scolladon that's correct; in our case the .email was 0 in size.
Would be good to recheck the non-zero .email filesize.
Thanks.

@github-actions
Copy link

Shipped in release v5.16.0.
You can install the new version using the version number or the latest-rc channel

$ sfdx plugins:install sfdx-git-delta@latest-rc
$ sfdx plugins:install [email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants