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

JSP Connector Update #195

Merged
merged 4 commits into from
Jul 12, 2017
Merged

JSP Connector Update #195

merged 4 commits into from
Jul 12, 2017

Conversation

gmkll
Copy link

@gmkll gmkll commented Jun 30, 2017

Fixing to API changes (current RFM version 2.5.x), bugfixing, updating, i.e. making more coherent to and partially following java connector using helper functions error/util. Main difference may be context and upload handling, file cleanup function, Java 7 and lower servlet API 2.5 requirements, consolidating process may start in a later step. Cleanup resources, changed package using gradle. TODOS cft. readme.

@psolom
Copy link
Owner

psolom commented Jun 30, 2017

Thanks a lot!
I can see you made updates for both /jsp and /java connectors. Awesome!
I talked to @fabriceci and he wish to discuss some points with you, since he is developer of /java connector.
Could you print your Skype id to speed up the communication?

@gmkll
Copy link
Author

gmkll commented Jun 30, 2017 via email

@psolom
Copy link
Owner

psolom commented Jun 30, 2017

I would prefer Skype. What time you are online in Skype generally?
Hope you will find a convenient way to chat

@gmkll
Copy link
Author

gmkll commented Jun 30, 2017

ok, I ´ll update my skype settings. Next week I am not online before Tuesday, otherwise afternoons are best, except may be Friday. CU

@psolom
Copy link
Owner

psolom commented Jul 1, 2017

Ok, me and @fabriceci have already sent request to you in Skype. Ping us once you are online, please.

@psolom psolom merged commit 012078d into psolom:master Jul 12, 2017
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