-
-
Notifications
You must be signed in to change notification settings - Fork 245
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
Replace deprecated OptionsMenu with MenuProvider #547
Comments
Assigning it to you |
Hi Varsha,
Lot of folks approaching me for working on NotyKT. Since you already worked
on couple of issues and feature, If it's okay for you, can I let someone
else to work on pin backend API integration in noty android app?
…On Thu, 6 Oct 2022, 4:52 pm Varsha Kulkarni, ***@***.***> wrote:
Replace the usages of
setHasOptionsMenu(Boolean): Unit
onCreateOptionsMenu(Menu, MenuInflater): Unit
onOptionsItemSelected(MenuItem): Boolean
onPrepareOptionsMenu(Menu): Unit
with MenuProvider
<https://developer.android.com/reference/kotlin/androidx/core/view/MenuProvider>
—
Reply to this email directly, view it on GitHub
<#547>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEVWFOEYWH3U7UEWKJPEGJTWB2ZARANCNFSM6AAAAAAQ6QGQZY>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Sure Shreyas, once I am done with this Options menu issue we will definitely go for more feature additions. It's a privilege that you are asking me this:) |
Hi Shreyas, I have worked on the same pin_notes branch so that all Options Menu related code gets modified. Is that okay? |
Yeah, works 👍 |
4 tasks
This was
linked to
pull requests
Oct 9, 2022
Repository owner
moved this from To do
to Done
in NotyKT
Oct 9, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Replace the usages of
with
MenuProvider
The text was updated successfully, but these errors were encountered: