Skip to content

Single tap top#108

Merged
aritra-tech merged 3 commits into
aritra-tech:developfrom
githubashutoshsoni:single_tap_top
Oct 4, 2023
Merged

Single tap top#108
aritra-tech merged 3 commits into
aritra-tech:developfrom
githubashutoshsoni:single_tap_top

Conversation

@githubashutoshsoni

Copy link
Copy Markdown

added scroll to top when notes item is clicked.

@githubashutoshsoni

Copy link
Copy Markdown
Author

@aritra-tech merge conflict is resolved now.

@githubashutoshsoni

Copy link
Copy Markdown
Author

Can you please accept this merge reqeuest?

@githubashutoshsoni

Copy link
Copy Markdown
Author

@aritra-tech You mentioned you added to the PR. But I don't see it anywhere. Can you please share again over here?

@aritra-tech

aritra-tech commented Oct 4, 2023

Copy link
Copy Markdown
Owner

@aritra-tech You mentioned you added to the PR. But I don't see it anywhere. Can you please share again over here?

I have added a code review, not the PR you can't see the review?

image

@githubashutoshsoni

Copy link
Copy Markdown
Author

Yes, I did not receive this notification anywhere in the github. I got the code changes so I will implement them. If possible can you share the link for review here so I can get access to it?

@githubashutoshsoni

Copy link
Copy Markdown
Author

Kindly add reviews that needs to be added over here. I've implemented the changes now @aritra-tech

@aritra-tech aritra-tech left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM 👍

Comment thread app/src/main/java/com/aritra/notify/navigation/NotifyApp.kt Outdated
@aritra-tech aritra-tech merged commit d424c4e into aritra-tech:develop Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants