-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[HOLD for payment 2021-12-27] Add support for Frequently/Recently Used Emojis #4559
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
Comments
Triggered auto assignment to @sonialiap ( |
Proposal We should have additional category It could have a set of default emojis in the list by default if we choose to keep Frequently used. Based on the user behavior we can sync this with Onyx and ensure that it available for the user to use. I don't think it is necessary to sync it across devices, but can be discussed. Changes: An additional
Extend
For frequently used, the logic would differ. We would have to somehow manage a counter { 1faa: 10, 1f78: 45, ...}. The logic of the map would also change as we would then add based on the counter. Apple adds to the list by using both - Recently used anf Frequently used. Your frequently used appear first and then your recently used. A combined list could also work for us. |
@sonialiap Huh... This is 4 days overdue. Who can take care of this? |
@sonialiap 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
Triggered auto assignment to @timszot ( |
This comment has been minimized.
This comment has been minimized.
Triggered auto assignment to @michaelhaxhiu ( |
@mananjadhav thank you for being patient on this :). The past 2 weeks have been abnormal for us, and we are just getting back to normal operations (especially true for me). |
Job posted here. https://www.upwork.com/jobs/~010528886766f948f5 |
@mananjadhav , I assigned issue to you, can you apply to the Upwork job so I can get you hired there? https://www.upwork.com/jobs/~01055c345a6223f23c Please confirm in a comment here once you have |
I think we can just do it every time someone sends an emoji, it's not that big of a deal IMO
I was suggesting we just run through the unsorted list and find the most used ones, but I changed my mind 😄 I think sorting like you suggested makes sense |
@mananjadhav Any progress here? :) |
All done, will raise Pr today |
@mananjadhav , when can we expect the PR? |
PR Raised. There was a small issue with EmojiPicker padding in iOS, which I thought was caused by my code. Turns out it exists in the main branch too. I've fixed that anyway with the current PR. |
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.1.21-1 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2021-12-27. 🎊 |
@mallenexpensify I was checking in my Upwork account I can't seem to find this job. I had applied to the job linked here #4559 (comment) but it says Job no longer available. |
Hey @mallenexpensify I think we should double the price of this to $500. The PR was solid and @mananjadhav put in a lot of work 😄 |
Thanks @stitesExpensify. Here's what I'm seeing for the breakdown
@mananjadhav does that look right? I created a new job, can you apply and confirm in a comment once you have? |
Thanks @stitesExpensify and @mallenexpensify. Applied on Upwork. |
Hired you @mananjadhav , add a comment here once you've accepted the offer and I'll pay. Thanks |
Accepted @mallenexpensify |
I just paid this, it looks like it's a few days early, we normally pay 7 days after code has hit production, if no regressions. I'm going to 🤞 and hope there are no regressions. If there are.. please reopen |
Yeah, I was just about to ping that it got paid early. I'll take up if there's any regression, |
I don't think you're going anywhere @mananjadhav . Congrats on being a top 3 contributor!! |
Thank you 😄 |
Uh oh!
There was an error while loading. Please reload this page.
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
With respect to comment #4456 (comment), adding this issue.
Action Performed:
Expected Result:
There should be an option to see Recently used or Frequently used Emojis.
Actual Result:
No option to use recently used/frequently used Emojis.

Workaround:
User has to find the Emoji and enter again
Platform:
Where is this issue occurring?
Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
View all open jobs on Upwork
Upwork job: https://www.upwork.com/jobs/~010528886766f948f5
The text was updated successfully, but these errors were encountered: