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

Deployment Problem might related to #166 #167

Closed
ncnxx opened this issue Aug 13, 2017 · 5 comments
Closed

Deployment Problem might related to #166 #167

ncnxx opened this issue Aug 13, 2017 · 5 comments

Comments

@ncnxx
Copy link
Contributor

ncnxx commented Aug 13, 2017

This is what I got when I am trying to deploy fflib-apex-common after the apex-mock. It might have something to do with #166 .

Deployment Started
Status: Queued
Status: InProgress
Status: Completed
Deployment Complete
Failures:
classes/fflib_SObjectMocks.cls(45,23):Class fflib_SObjectMocks.SObjectUnitOfWork must implement the method: void fflib_ISObjectUnitOfWork.registerRelationship(Messaging.SingleEmailMessage, SObject)
classes/fflib_ApplicationTest.cls(45,23):Dependent class is invalid and needs recompilation:
Class fflib_SObjectMocks : Class fflib_SObjectMocks.SObjectUnitOfWork must implement the method: void fflib_ISObjectUnitOfWork.registerRelationship(Messaging.SingleEmailMessage, SObject)
fflib/src
fflib/src/package.xml
fflib/src/classes

@ncnxx
Copy link
Contributor Author

ncnxx commented Aug 13, 2017

I have tried deploying commit no. 7366576
and it worked. So, the new pull request is the problem.

@martykopka
Copy link

martykopka commented Aug 20, 2017

Im getting the same problem when trying to do the trailhead and deploy .I was just pushing the deploy button. How can I deploy that particular commit?

@martykopka
Copy link

Its okay i worked it out. IN the deploy tool on heroku just need to use: 7366576
as the commit value to deploy. That worked for me and was now able to deploy.

@ncnxx
Copy link
Contributor Author

ncnxx commented Sep 2, 2017

The pull request fixed the problem. This issue can be closed.

@ncnxx ncnxx closed this as completed Sep 2, 2017
@afawcett
Copy link
Contributor

@martykopka thats cool, didn't now the tool supported that! 👍

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

No branches or pull requests

3 participants