Repository navigation
[$500] Private notes - Note editor opens instead of list for user with existing private notes #32302
Description
Activity
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
- Identify the pull request that introduced this issue and revert it.
- Find someone who can quickly fix the issue.
- Fix the issue yourself.
Triggered auto assignment to @chiragsalian (
Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.@lanitochka17, can you elaborate what you mean by this,
Open the chat from the precondition
what do you mean by precondition?
Also i tested the private notes on a DM and a group chat and it seemed to work fine for me on staging chrome. Can you double check?
Meanwhile I'll ping the authors of #31344 to double check as well.As discussed here, not a blocker.
Assigning @DylanDylann and @ArekChr to follow up with a fix as early as possible.
If it helps, i was able to reproduce the issue by logging out, logging in and then clicking on an existing private notes. Then i see the editor instead of the list. For me this happens just once and then its fine until i log out and log in again. Please check and implement a fix.- changed the title
[-]Private notes - Note editor opens instead of list for user with existing private notes[/-][+][$500] Private notes - Note editor opens instead of list for user with existing private notes[/+]on Nov 30, 2023 16 remaining items
@jjcoffee Can you help check this one #32302 (comment) once you have a chance
Issue not reproducible during KI retests. (First week)
@DylanDylann I'm not sure I fully understand your comment or if you are proposing a particular solution, but I think it's up to the C+ on this issue to handle regressions. cc @ArekChr
Hey, sorry for the late reply. I wanted to let you know that I'm no longer in the C+ role, as I've moved to the Waves team. Could you please assign a new C+ to take over here? Thanks!
@mallenexpensify This seems to require a new C+ for some time, and Overdue was not applied here due to the Reviewing label. Can QA retest #32302 (comment) and maybe we can open this for External contributors? I am happy to volunteer for C+ position here.
Reacted by Matt Allen@chiragsalian, @ArekChr, this Monthly task hasn't been acted upon in 6 weeks; closing.
If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.
QA, for retesting, here's a vid example
#32302 (comment)@shubham1206agra , yes, I'll assign you as C+ if the bug persists after more testing. Did you try to reproduce?
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.6-2
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR #31344
Action Performed:
Precondition: There is a chat with some content in the Privates notes and the note is not visited before. If visited, please log out and log in again
Expected Result:
User is navigated to private note list screen
Actual Result:
User is navigated to private note editor instead of private note list screen
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6296532_1701375305114.bandicam_2023-12-01_02-53-05-577__1_.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit