Skip to content
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

[[ Bug 20551 ]] Update Android Notification module to compile against API 23#6558

Merged
livecodepanos merged 3 commits into
livecode:develop-9.0from
livecodeian:bugfix-20551
Jun 2, 2018
Merged

[[ Bug 20551 ]] Update Android Notification module to compile against API 23#6558
livecodepanos merged 3 commits into
livecode:develop-9.0from
livecodeian:bugfix-20551

Conversation

@livecodeian

Copy link
Copy Markdown
Contributor

No description provided.

@livecodeian livecodeian added this to the 9.0.1-rc-1 milestone Jun 1, 2018

t_builder.setContentIntent(createNotificationContentIntent(context));
t_builder.setContentText(p_body);
t_builder.setContentTitle(p_title);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You've set the title twice :-)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've deleted the duplicate

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heh, great commit

@livecodeali

Copy link
Copy Markdown
Member

@livecode-vulcan review ok 637ebd6

@livecode-vulcan

Copy link
Copy Markdown
Contributor

💙 review by @livecodeali ok 637ebd6

livecode-vulcan added a commit that referenced this pull request Jun 1, 2018
[[ Bug 20551 ]] Update Android Notification module to compile against API 23
@livecode-vulcan

Copy link
Copy Markdown
Contributor

😞 test failure 637ebd6

  • try-community-armv6-android-api8: success
  • try-community-armv6-android-api9: success
  • try-community-js-emscripten-sdk1.35: success
  • try-community-universal-ios-iphoneos11.4: success
  • try-community-universal-ios-iphonesimulator11.4: success
  • try-community-universal-mac-macosx10.6: success
  • try-community-universal-mac-macosx10.9: success
  • try-community-x86-linux-debian7: success
  • try-community-x86-linux-debian8: success
  • try-community-x86_64-linux-debian7: success
  • try-community-x86_64-linux-debian8: success
  • try-community-x86-win32: success
  • try-community-x86_64-win32: failure

@livecodepanos

Copy link
Copy Markdown
Contributor

@livecode-vulcan retry 637ebd6

livecode-vulcan added a commit that referenced this pull request Jun 2, 2018
[[ Bug 20551 ]] Update Android Notification module to compile against API 23
@livecode-vulcan

Copy link
Copy Markdown
Contributor

😎 test success 637ebd6

  • try-community-armv6-android-api8: success
  • try-community-armv6-android-api9: success
  • try-community-js-emscripten-sdk1.35: success
  • try-community-universal-ios-iphoneos11.4: success
  • try-community-universal-ios-iphonesimulator11.4: success
  • try-community-universal-mac-macosx10.6: success
  • try-community-universal-mac-macosx10.9: success
  • try-community-x86-linux-debian7: success
  • try-community-x86-linux-debian8: success
  • try-community-x86_64-linux-debian7: success
  • try-community-x86_64-linux-debian8: success
  • try-community-x86-win32: success
  • try-community-x86_64-win32: success

@livecodepanos livecodepanos merged commit 980dc97 into livecode:develop-9.0 Jun 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants