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

[$250] iOS - Live MD - Live markdown preview is not displayed on iOS 15 #40908

Closed
1 of 6 tasks
lanitochka17 opened this issue Apr 24, 2024 · 30 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@lanitochka17
Copy link

lanitochka17 commented Apr 24, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 1.4.65-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4497423
Issue reported by: Applause - Internal Team

Action Performed:

  1. Log in to New Expensify
  2. Navigate to any chat
  3. Draft a message for with any markdown

Expected Result:

Live markdown preview should be displayed in compose box

Actual Result:

Markdown preview in compose box is not displayed at all
Issue is also reproducible in task, IOU, workspace descriptions
Reproducible since v1.4.63-0 on iPhone 7/iOS 15.8.2 and iPhone 8/iOS 15.7

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6453800_1713455035622.JUUR8323.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~010578c1fe18adb6be
  • Upwork Job ID: 1792329725805604864
  • Last Price Increase: 2024-05-29
  • Automatic offers:
    • rojiphil | Reviewer | 102890946
Issue OwnerCurrent Issue Owner: @
@lanitochka17 lanitochka17 added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Apr 24, 2024
Copy link

melvin-bot bot commented Apr 24, 2024

Triggered auto assignment to @mallenexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@lanitochka17
Copy link
Author

@mallenexpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@lanitochka17
Copy link
Author

We think that this bug might be related to #vip-vsp

@mallenexpensify
Copy link
Contributor

Unable to repro on latest staging version for iOS native - v1.4.66-2

RPReplay_Final1714090371.MP4

@mallenexpensify mallenexpensify added Weekly KSv2 retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Needs Reproduction Reproducible steps needed and removed Daily KSv2 labels Apr 26, 2024
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@melvin-bot melvin-bot bot added the Overdue label May 6, 2024
@mallenexpensify
Copy link
Contributor

Waiting for retest. Just tried again to repro and wasnt able to

@melvin-bot melvin-bot bot removed the Overdue label May 7, 2024
Copy link

melvin-bot bot commented May 8, 2024

@mallenexpensify this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@melvin-bot melvin-bot bot added the Overdue label May 16, 2024
@mvtglobally
Copy link

Issue is reproducible during KI retests.

CLAN7016.MP4

@mvtglobally mvtglobally removed the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label May 17, 2024
@mallenexpensify mallenexpensify added the Internal Requires API changes or must be handled by Expensify staff label May 19, 2024
Copy link

melvin-bot bot commented May 19, 2024

Job added to Upwork: https://www.upwork.com/jobs/~010578c1fe18adb6be

Copy link

melvin-bot bot commented May 19, 2024

Triggered auto assignment to Contributor Plus for review of internal employee PR - @rojiphil (Internal)

@melvin-bot melvin-bot bot removed the Overdue label May 19, 2024
@mallenexpensify
Copy link
Contributor

Weird QA is able to repro. I'm not on an iphone 15 via Browserstack, nor on my iPhone X. ¯_(ツ)_/¯
@rojiphil can you test?

2024-05-19_15-59-36.mp4

@melvin-bot melvin-bot bot added the Overdue label May 28, 2024
@mallenexpensify
Copy link
Contributor

@rojiphil , plz test and let me know if you can repro

@rojiphil
Copy link
Contributor

@mallenexpensify I can reproduce this on iOS 15.5 (iPhone 13 Pro) simulator.
Here is a test video for the same.

40908-ios-15-repro-01.mp4

@melvin-bot melvin-bot bot removed the Overdue label May 29, 2024
@mallenexpensify mallenexpensify removed the Internal Requires API changes or must be handled by Expensify staff label May 29, 2024
Copy link

melvin-bot bot commented Jun 10, 2024

@rojiphil, @mallenexpensify Huh... This is 4 days overdue. Who can take care of this?

@mallenexpensify
Copy link
Contributor

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jun 12, 2024
@mallenexpensify mallenexpensify added Weekly KSv2 and removed Daily KSv2 labels Jun 14, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jun 14, 2024
@mallenexpensify
Copy link
Contributor

Still chatting about markdown-related issues here

@mallenexpensify
Copy link
Contributor

Waiting for it to be assigned/worked on/ prioritized.

@melvin-bot melvin-bot bot removed the Overdue label Jun 26, 2024
@tomekzaw
Copy link
Contributor

I've managed to reproduce this bug in react-native-live-markdown exmaple app running on iPhone 13 Pro (iOS 15.5):

Reported in react-native-live-markdown as Expensify/react-native-live-markdown#400

Screenshot 2024-06-26 at 13 41 13

@tomekzaw
Copy link
Contributor

tomekzaw commented Jun 26, 2024

@tomekzaw
Copy link
Contributor

Should be fixed with Expensify/react-native-live-markdown#83.

This comment was marked as off-topic.

@melvin-bot melvin-bot bot added the Overdue label Jul 5, 2024
@mallenexpensify
Copy link
Contributor

@tomekzaw is this fixed and working? It looks like it should be from

@melvin-bot melvin-bot bot removed the Overdue label Jul 10, 2024
@tomekzaw
Copy link
Contributor

tomekzaw commented Jul 10, 2024

@mallenexpensify The PR to react-native-live-markdown you've mentioned (Expensify/react-native-live-markdown#404) was released in 0.1.96 but E/App is still on 0.1.91 (see here).

We need to merge this bump PR to E/App:

@melvin-bot melvin-bot bot added the Overdue label Jul 18, 2024
@mallenexpensify
Copy link
Contributor

@melvin-bot melvin-bot bot removed the Overdue label Jul 24, 2024
@BartoszGrajdek BartoszGrajdek moved this from HIGH to Done in Live Markdown Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
Status: Done
Development

No branches or pull requests

7 participants