Skip to content
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

[NotificationCenter] Bindings for iOS Xcode 11 beta 1 #6579

Merged
merged 15 commits into from Jul 16, 2019

Conversation

joconte
Copy link
Contributor

@joconte joconte commented Jul 13, 2019

I decided to give a try to Notification Center Bindings for iOS beta 1

diff : https://github.com/xamarin/xamarin-macios/wiki/NotificationCenter-iOS-xcode11-beta1

@joconte joconte requested a review from spouliot as a code owner July 13, 2019 12:25
src/notificationcenter.cs Outdated Show resolved Hide resolved
src/notificationcenter.cs Outdated Show resolved Hide resolved
src/notificationcenter.cs Outdated Show resolved Hide resolved
Copy link
Member

@rolfbjarne rolfbjarne left a comment

Choose a reason for hiding this comment

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

src/notificationcenter.cs Outdated Show resolved Hide resolved
src/notificationcenter.cs Outdated Show resolved Hide resolved
joconte and others added 3 commits July 15, 2019 11:16
Co-Authored-By: Rolf Bjarne Kvinge <rolf@xamarin.com>
Change deprecated message because I changed the method name
src/notificationcenter.cs Show resolved Hide resolved
@joconte
Copy link
Contributor Author

joconte commented Jul 15, 2019

@rolfbjarne @spouliot
Thanks to both of you, I will fix it after work.

Copy link
Member

@rolfbjarne rolfbjarne left a comment

Choose a reason for hiding this comment

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

build

@dalexsoto dalexsoto added community Community contribution ❤ note-highlight Worth calling out specifically in release notes labels Jul 15, 2019
@dalexsoto
Copy link
Member

build

@monojenkins
Copy link
Collaborator

Build failure
Build succeeded
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
ℹ️ Generator Diff (please review changes)
🔥 Test run failed 🔥

Test results

1 tests failed, 90 tests passed.

Failed tests

  • Xtro/Mac: Failed (Test run failed.)

@mandel-macaque
Copy link
Member

@joconte missing a selector on iOS UIVibrancyEffect::widgetEffectForVibrancyStyle

@mandel-macaque mandel-macaque changed the title [xcode11] Notification Center Bindings for iOS beta 1 [NotificationCenter] r Bindings for iOS beta 1 Jul 16, 2019
@mandel-macaque mandel-macaque changed the title [NotificationCenter] r Bindings for iOS beta 1 [NotificationCenter] Bindings for iOS Xcode 11 beta 1 Jul 16, 2019
src/notificationcenter.cs Show resolved Hide resolved
@joconte
Copy link
Contributor Author

joconte commented Jul 16, 2019

build

1 similar comment
@chamons
Copy link
Contributor

chamons commented Jul 16, 2019

build

@monojenkins
Copy link
Collaborator

Build success
Build succeeded
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
ℹ️ Generator Diff (please review changes)
Test run succeeded

@spouliot spouliot merged commit 9b7f6f9 into xamarin:xcode11 Jul 16, 2019
@spouliot
Copy link
Contributor

Thanks again! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Community contribution ❤ note-highlight Worth calling out specifically in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants