Skip to content
This repository was archived by the owner on Jun 16, 2024. It is now read-only.

Added DeleteRecordChangeHistoryRequest Fake executor for Shared project.#542

Open
victorsolaya wants to merge 4 commits intojordimontana82:masterfrom
victorsolaya:AddDeleteRecordChangeHistoryRequestFakeMessage
Open

Added DeleteRecordChangeHistoryRequest Fake executor for Shared project.#542
victorsolaya wants to merge 4 commits intojordimontana82:masterfrom
victorsolaya:AddDeleteRecordChangeHistoryRequestFakeMessage

Conversation

@victorsolaya
Copy link

Title

Added DeleteRecordChangeHistoryRequestTest Fake executor for Test Shared project.

Description

It has been included a quick fake DeleteRecordChangeHistory to fake the message in the Shared Projects.
It has been created a DeleteRecordChangeHistory_Tests as well in the SharedTest project.

Victor Sanchez added 2 commits January 8, 2021 21:43
@victorsolaya
Copy link
Author

Hi @jordimontana82 I am having some errors because DeleteRecordChangeHistory was added in the v9.0.1.0

9.0.1.0: UR1 (Update Rollup 1)
   Added DeleteRecordChangeHistory message to delete audit change history for a particular record

How could I say to my tests not to include this if it doesn't exist? Do you have any example?
CC @BetimBeja just in case you had the same issue before and you could help me.

@BetimBeja
Copy link
Contributor

I made a PR victorsolaya#1 to your repository which will update this PR if merged.
Each build of FakeXrmEasy defines a symbol for the dependency it has on the specific version of the CrmSdk. When there are braking changes we can use those symbols in the #if directives to handle the missing dependencies.

…istory

Added preprocessor directive to exclude older version
@jordimontana82
Copy link
Owner

Thanks both for this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants