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

Created new parameter syncCall for RFWebServiceCall attribute #341

Closed
wants to merge 12 commits into from
Closed

Created new parameter syncCall for RFWebServiceCall attribute #341

wants to merge 12 commits into from

Conversation

AlexeyAfanasyev
Copy link
Contributor

Created new parameter syncCall for RFWebServiceCall attribute that force web service request to perform synchronously with the thread it was created from.

# By Yuru Taustahuzau (3) and Evgenii Kamyshanov (2)
# Via Yuru Taustahuzau (5) and Evgenii Kamyshanov (1)
* 'master' of https://github.com/epam/road-ios-framework:
  Removing obsolete info from web service docs
  NSMutableString(RFStringFormatter) refactoring
  Adding concurentAuth method to public interface #329
  Fixing implicit body passing for PUT method #332
  Minor fixes for code review #66

Conflicts:
	Framework/ROADCore/ROADCore.xcodeproj/project.pbxproj
	Framework/ROADWebService/ROADWebService.xcodeproj/project.pbxproj
	Framework/ROADWebService/ROADWebserviceTest/Classes/RFConcreteWebServiceClient.h
	Framework/ROADWebService/ROADWebserviceTest/RFWebServiceTest.m
2) cheating Simian
outputPaths = (
);
runOnlyForDeploymentPostprocessing = 0;
shellPath = /bin/sh;
shellScript = "\"${SRCROOT}/../Pods/Pods-ROADCore-resources.sh\"\n";
showEnvVarsInLog = 0;
shellScript = "\"${PODS_ROOT}/libObjCAttr/tools/binaries/ROADAttributesCodeGenerator\" -src=\"${SRCROOT}/${TARGET_NAME}\" -src=\"${PODS_ROOT}\" -dst=\"${SRCROOT}/${TARGET_NAME}/ROADGeneratedAttributes/\"";
Copy link
Contributor

Choose a reason for hiding this comment

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

@AlexeyAfanasyev Why do we have ${SRCROOT}/${TARGET_NAME}. Please, replace it with just ${SRCROOT}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's not me, it's ROADConfigurator.rb

@yuriy-tolstoguzov
Copy link
Contributor

@AlexeyAfanasyev Test for sync call fails too. Please, fix it.

@yuriy-tolstoguzov
Copy link
Contributor

@AlexeyAfanasyev also, when you will be done, please, squash all these commits into 1

@yuriy-tolstoguzov
Copy link
Contributor

Changes merged within #347

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.

None yet

2 participants