rainbow icon indicating copy to clipboard operation
rainbow copied to clipboard

Upgrade kitura

Open sanjeevghimire opened this issue 6 years ago • 7 comments

  • upgrade kitura
  • new CFC checklist items and model

sanjeevghimire avatar May 13 '19 20:05 sanjeevghimire

Daaaaaaaang this pull request is large

dokun1 avatar May 13 '19 20:05 dokun1

+27,444 −318 NBD

stevemar avatar May 13 '19 20:05 stevemar

I think its because of the upgrade and changes are there on both iOS and Server.

sanjeevghimire avatar May 13 '19 21:05 sanjeevghimire

I promise to check this by the end of the week. On a side thread, I absolutely will help you get this submitted to the app store tomorrow, but let's take a second look at this and see what we can cull off.

dokun1 avatar May 14 '19 00:05 dokun1

Both app and server are thoroughly tested. There are few things I have removed which I have enhanced a bit. We can go over that tomorrow, test it out one more time and push.

On Mon, May 13, 2019 at 7:13 PM David Okun [email protected] wrote:

I promise to check this by the end of the week. On a side thread, I absolutely will help you get this submitted to the app store tomorrow, but let's take a second look at this and see what we can cull off.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/IBM/rainbow/pull/91?email_source=notifications&email_token=AAM5RWZFN2JEQCQ572U7PDTPVH7Y5A5CNFSM4HMTFUA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVJ4YTY#issuecomment-492031055, or mute the thread https://github.com/notifications/unsubscribe-auth/AAM5RWZ5Z7NKONZN7BBOBB3PVH7Y5ANCNFSM4HMTFUAQ .

-- सन्जीब िघमिरे (Sanjeev Ghimire)

sanjeevghimire avatar May 14 '19 00:05 sanjeevghimire

nice

stevemar avatar May 16 '19 18:05 stevemar

JFYI we have released v1.2 of this app to the app store 🎉

Neil found a bug that I've already fixed with a commit here, and I'm testing to see if there's anything else I can find wrong with it. The bug is not a showstopper, so this version being live is ok, but I intend to review this PR entirely today, and that way 1.2.1 can be released from the master branch.

dokun1 avatar May 20 '19 16:05 dokun1