Comments (8)
jesperfj
commented on July 17, 2024
Thanks for the suggestion.
Is it worth considering supporting JodaTime explicitly (with a toggle or auto-detected if possible)? I'd like to keep the library as simple as possible in terms of surface area.
But I admit I haven't thought deeply about this. Do you have a code sample of how you want it to work ideally?
from force-rest-api.
jesperfj
commented on July 17, 2024
Hm. I guess JodaTime is discouraged after JDK 8 which has already been out there for a while now. Then it should probably not be explicitly supported.
My concern with opening up for customizing objectmappers and such is that it makes it harder to evolve this library with backwards compatibility in the future.
from force-rest-api.
bcallender
commented on July 17, 2024
That makes sense -- I've migrated my project over to the Java built in objects since I can use Java 8. It is worth noting though that a lot of people using Salesforce are enterprise and may be limited to Java 7 though.
I think you still need to install the Jackson modules for Java 8 if you want to support Java 8's LocalDateTime and ZonedDateTime serialization/deserialization. I'm trying to use ZonedDateTime and the library isn't serializing it to an ISO 8601 value like the Salesforce API wants.
from force-rest-api.
jesperfj
commented on July 17, 2024
Ok. I'll look into Jackson config when I find some time (if you don't beat
me to it).
I won't worry about developers being stuck on 7 until I hear complaints
:-).
…On Tue, May 30, 2017 at 3:33 PM Brandon Callender ***@***.***> wrote:
That makes sense -- I've migrated my project over to the Java built in
objects since I can use Java 8. It is worth noting though that a lot of
people using Salesforce are enterprise and may be limited to Java 7 though.
I think you still need to install the Jackson modules for Java 8
<https://github.com/FasterXML/jackson-modules-java8> if you want to
support Java 8's LocalDateTime and ZonedDateTime
serialization/deserialization.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#45 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AARUrFCc-73VkyKgHbdYW6EtXzroiL7pks5r_JlCgaJpZM4NnAH_>
.
from force-rest-api.
madorb
commented on July 17, 2024
I'm running into this same issue as well. I think that it's fairly standard practice to expose the JSON Serializers for external configuration - joda time is just one of many reasons that folks may want t configure that serialization. And for better or worse, a LOT of large enterprises (which have a fairly high likelihood to also be using salesforce) are still on older versions of java. I'm working for a large financial company atm and i'm limited to Java 7.
from force-rest-api.
jesperfj
commented on July 17, 2024
@madorb Take a look at #51 and see if this would work for you. Specifically look at JodaTimeSample
from force-rest-api.
jesperfj
commented on July 17, 2024
@madorb @bcallender check out release 0.0.38. You can set a custom ObjectMapper on ApiConfig now.
from force-rest-api.
jesperfj
commented on July 17, 2024
I'll close this issue for now. Feel free to reopen if 0.0.38 does not address your needs.
from force-rest-api.
Related Issues (20)
- Add formula to DescribeSObject.Field
- Force REST API - giving below exception
- BUG with get knowledgeArticles HOT 3
- Return http responses instead of void calls
- updateSObject: "The Id field should not be specified in the sobject data" HOT 1
- Missing instance_url and id in ApiSession HOT 2
- Support for jdk 10 HOT 1
- Support for Blob data HOT 2
- Redirect not working HOT 1
- did not save the the id of the created object. any way to update the object?
- Test failure on v46 HOT 2
- Support for Salesforce Platform Events HOT 1
- Adding support for batch size for SOQL HOT 5
- Provide support for Rest Resource calls. HOT 1
- JDK 8, mvn install -DskipTests is failing. HOT 1
- Username and password are not escaped in soap login.
- Is support search api now? HOT 4
- Add support for Headers HOT 1
- autoRenew in ForceAPI.java not working
- Add support for JWT Bearer Flow HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from force-rest-api.