Git Product home page Git Product logo

Comments (6)

mrmcduff-stripe avatar mrmcduff-stripe commented on May 31, 2024

If you prefer to keep that change out, you can roll the change back in your code by using
compile 'com.stripe:stripe-android:1.0.5', which is the version before any of the changes took place.

Since I don't otherwise use TextUtils, I could use something else in a future update for this check, but use of the android framework is probably going to creep into new features at some point.

from stripe-android.

mrmcduff-stripe avatar mrmcduff-stripe commented on May 31, 2024

Since you can fix the broken tests using a quick gradle change, I'm going to go ahead and close this issue. Let me know if rolling back the version in your code doesn't work!

from stripe-android.

nostra13 avatar nostra13 commented on May 31, 2024

We use Stripe 1.0.6 now and it works (it has no Android framework dependency yet) but we obviously want to get new features/fixes in future Stripe releases. So dependency on Android framework makes our Android-independent Unit testing of payment logic impossible.

I think you should avoid Android-framework dependency as wide as you can to allow developers to use the library in Unit tests. As I see it's possible for now, just need to rollback boolean isWholePositiveNumber(String value) method.

I wonder if it's gonna happen and should I looking forward for 1.1.1 soon?

from stripe-android.

mrmcduff-stripe avatar mrmcduff-stripe commented on May 31, 2024

Are you trying to share code between an android and a java application? I'm a little confused as to why you'd want to write an Android app without the Android framework.

from stripe-android.

mrmcduff-stripe avatar mrmcduff-stripe commented on May 31, 2024

It's not a big deal for me to move it out of that method - I added it by habit more than anything else. But I hadn't planned to make a concerted effort to keep core android classes out of future updates to our android API. There are future plans down the line to do things like having the simple checkout forms, similar to what exists in the stripe-ios SDK, and creating overlay forms is something I'd definitely use many core classes for.

from stripe-android.

mrmcduff-stripe avatar mrmcduff-stripe commented on May 31, 2024

I'll go ahead and take this out for now -- but I'm doubtful on my ability to keep the android framework out of the API indefinitely.

from stripe-android.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo 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.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.