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

Sm/event-task-custom-fields #348

Merged
merged 19 commits into from
Aug 2, 2023
Merged

Conversation

mshanemc
Copy link
Contributor

Event/Task in profiles now match Activity in (package.xml).

rewrite of the Profile Rewrite process

  • it's now extracted as a testable function. Pass in a profile (in JSON) and a map representation of the package.xml contents, get back what the profile would be
  • uses type-safe Profile metadata information from jsforce
  • those types allow manipulation of json properties instead of using DOM tooling to deal with XML
  • moved logging of Profile modifications from a separate ProfileInformation class to inline (let logger handle level decisions
  • removed some unused dependencies

forcedotcom/cli#2278
@W-13753544@

Copy link
Contributor

@shetzel shetzel left a comment

Choose a reason for hiding this comment

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

Nice! Requested some small changes.

types: Array<{ name: string; members: string[] }>;
version: string;
};
};
Copy link
Contributor

Choose a reason for hiding this comment

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

This type isn't defined/exported in SDR already?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

defined, yes. Exported, not at the top-level.

I'm going to get it from jsforce

src/package/packageProfileApi.ts Outdated Show resolved Hide resolved
src/package/profileRewriter.ts Show resolved Hide resolved
Copy link
Member

@WillieRuemmele WillieRuemmele left a comment

Choose a reason for hiding this comment

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

Agree with Steve's comments

src/package/packageProfileApi.ts Outdated Show resolved Hide resolved
src/package/packageVersionCreate.ts Outdated Show resolved Hide resolved
src/package/packageProfileApi.ts Outdated Show resolved Hide resolved
@mshanemc mshanemc marked this pull request as ready for review July 26, 2023 17:10
ignoreDeclaration: true,
numberParseOptions: { leadingZeros: false, hex: false },
// make sure types and members is always an array
isArray: (name: string) => ['types', 'members'].includes(name),
Copy link
Member

Choose a reason for hiding this comment

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

nice they provide this

Copy link
Contributor

@amtrack amtrack left a comment

Choose a reason for hiding this comment

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

applicationVisibilities were removed incorrectly:

The "userPermissions" profile setting was removed from the "REDACTED Profile" profile.

The metadata type is CustomApplication not Application.

mshanemc and others added 2 commits August 2, 2023 14:57
Co-authored-by: Matthias Rolke <mr.amtrack@gmail.com>
@WillieRuemmele WillieRuemmele merged commit ebf7888 into main Aug 2, 2023
9 checks passed
@WillieRuemmele WillieRuemmele deleted the sm/event-task-custom-fields branch August 2, 2023 22:27
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.

5 participants