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

Added initial implementation of DynamicsCRMHelper #1009

Merged
merged 1 commit into from
Dec 1, 2015

Conversation

DigitalFlow
Copy link
Contributor

Hey all,

I'm currently implementing a new helper for Dynamics CRM.
The use of it is to be able to export solutions (update component containers) from Dynamics CRM and being able to store those solutions in a VCS or even import them before doing unit tests for ensuring that the system is in the correct state for doing the unit tests.
Tests as well as a possibility for downlevelling the solution archive into xml representation will follow.

Kind Regards,
DigitalFlow

@DigitalFlow
Copy link
Contributor Author

What I was wondering about:
Is it ok that I added the NuGet Packages for CRM SDK? I use them for connecting to the CRM services.

@forki
Copy link
Member

forki commented Nov 18, 2015

Aehm that's not a good idea

@DigitalFlow
Copy link
Contributor Author

Ok, so I'll remove those packages and connect differently. I wonder why the mono build on Travis is failing, while the AppVeyor build with MSBuild works...

@DigitalFlow
Copy link
Contributor Author

@forki: How sounds using Fsharp Data TypeProviders for connecting to the CRM to you? Seems if I'd need them as NuGet packages too however.
https://www.nuget.org/packages/FSharp.Data/2.2.5/
https://www.nuget.org/packages/FSharp.Data.TypeProviders/

@forki
Copy link
Member

forki commented Nov 18, 2015

we do we need to add packages at all?

@DigitalFlow
Copy link
Contributor Author

Ok so I checked my options and I don't see an alternative to the CRM packages that does not produce a lot of overhead for me.
Shall we close the Pull Request in this case?

@forki
Copy link
Member

forki commented Nov 18, 2015

mhm. I think we should create an addition project then. We already have way to much stuff in this repo.

@DigitalFlow
Copy link
Contributor Author

All right then

@forki
Copy link
Member

forki commented Nov 18, 2015

sorry. Is there a command line version we can call from here?

@DigitalFlow
Copy link
Contributor Author

I just began developing a command line version, I'll check with my boss whether I am allowed to open source it. Are you thinking of making it a complete standalone tool, or a subproject of FAKE?

@forki
Copy link
Member

forki commented Nov 18, 2015

What we usually do is to create only a wrapper around the command line in fake (That stuff that calls the command line tool),

@DigitalFlow
Copy link
Contributor Author

Yes thats what I did for the SolutionPackager part, but I guess you mean for this whole module then.
I'll have to develop a command line version anyhow, I would just have to check with my boss for open sourcing it and writing a FAKE wrapper.

@DigitalFlow
Copy link
Contributor Author

Ok so it goes on. I am allowed to open source the tool.
The plan is to leave the solution packager part of this pull request as it is, because it does not need any packages. For the importing and exporting I will write a small command line tool and a wrapper for FAKE so that we can use the wrapper and don't need any nuget packages in here.

@DigitalFlow
Copy link
Contributor Author

Ok, my command line tool is done for now: https://github.com/DigitalFlow/Dynamics-CRM-Solution-Exchanger
The wrapper is already working in this PR, I already set up a build in our CI which exports a solution from CRM every night and pushes it into our VCS.

@DigitalFlow
Copy link
Contributor Author

@forki: Can you imagine why the build is failing? Local builds run just fine, I didn't add any assemblies that would need to be referenced and the previous build passed at least for the previous commit, while I only changed an XML tag in the current commit...

@forki
Copy link
Member

forki commented Nov 19, 2015

seems unrelated. mono fails with mono 4.2 issue and appveyor with nuget.

@DigitalFlow DigitalFlow changed the title [WIP] Added initial implementation of DynamicsCRMHelper Added initial implementation of DynamicsCRMHelper Nov 19, 2015
@@ -0,0 +1,143 @@
[<AutoOpen>]
Copy link
Member

Choose a reason for hiding this comment

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

please remove the autoopen

@DigitalFlow
Copy link
Contributor Author

Ok, I removed the auto open tag.
I also just rebased onto the newest master commit and into one single commit.

@DigitalFlow DigitalFlow force-pushed the feature/DynamicsCRMHelper branch 3 times, most recently from c7acf06 to 869a2a4 Compare November 30, 2015 17:04
@DigitalFlow
Copy link
Contributor Author

Ok, so I just added some documentation and some comments.
Do you need anything else for it being merged?

forki added a commit that referenced this pull request Dec 1, 2015
Added initial implementation of DynamicsCRMHelper
@forki forki merged commit 5d58de0 into fsprojects:master Dec 1, 2015
@forki
Copy link
Member

forki commented Dec 1, 2015

cool. thx

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