-
Notifications
You must be signed in to change notification settings - Fork 10
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
salesforce: move toUTF8
out to utils
#877
Comments
@mtuchi Tempted to do this for the next major release - what do you think? |
@josephjclark I think there is a dependence here, for this to work we will need to solve first |
I wasn't proposing to do this in common but actually that's a much better idea! |
@mtuchi I've hit a problem on moving toUTF8 to common toUTF8 depends on This is fine for salesforce but not possible for common. So here are our options:
I guess my question is - have you ever needed this function outside of salesforce? I would go with option 2 so long as you can guarantee me that this is actually useful |
@josephjclark i haven't come across a use-case for |
OK cool - for the moment I'll leave this as a salesforce util function. I am happy enough with 2 but it takes a bit more work and I'm reluctant to do it unless there's a good reason :) Thanks for the input! |
It's not an operation. We should lean into the idea that all top-level functions are Operations, and everything else goes on a namespaced object (like
utils.toTUF8
)The text was updated successfully, but these errors were encountered: