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

[HOLD for payment 2021-11-04] Chat - Chronos message is cut off at the edge and not fully visible #5008

Closed
kavimuru opened this issue Sep 2, 2021 · 31 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@kavimuru
Copy link

kavimuru commented Sep 2, 2021

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Lunch the app
  2. Log in with expensifail account
  3. On Search box - find Chronos
  4. Text - Top

Expected Result:

Chronos messages are fully visible.

Actual Result:

Chronos message isn't fully visible, it is cut off at the end

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • iOS
  • Android
  • Mobile Web

Version Number:
1.0.91-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Bug5219680_Screenshot_20210901-160649_New_Expensify

Expensify/Expensify Issue URL:

View all open jobs on GitHub

@MelvinBot
Copy link

Triggered auto assignment to @francoisl (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@francoisl francoisl removed their assignment Sep 2, 2021
@francoisl francoisl added the External Added to denote the issue can be worked on by a contributor label Sep 2, 2021
@MelvinBot MelvinBot added Weekly KSv2 and removed Daily KSv2 labels Sep 2, 2021
@MelvinBot
Copy link

Triggered auto assignment to @Christinadobrzyn (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@MelvinBot MelvinBot added Daily KSv2 and removed Weekly KSv2 labels Sep 2, 2021
@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Sep 3, 2021

External job created in Upwork - https://www.upwork.com/jobs/~0135dabda6dbb82f78
Internal job created in Upwork - https://www.upwork.com/ab/applicants/1433592402430713856/job-details

@Christinadobrzyn Christinadobrzyn added Exported Weekly KSv2 and removed Daily KSv2 labels Sep 3, 2021
@MelvinBot MelvinBot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 3, 2021
@MelvinBot
Copy link

Triggered auto assignment to @johnmlee101 (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@equationalapplications
Copy link

I launched the web app in my development environment and logged in with my email. I clicked the search icon, but I can't find Chronos.

@pROFESOR11
Copy link
Contributor

Hi @kavimuru ,
What exactly is expensifail account and how can I login with that?
I guess I need to login with that specific account to be able to reproduce the issue.

@Christinadobrzyn
Copy link
Contributor

Hi @pROFESOR11! Sorry for the delayed response, Expensifail is our test account. You can create a test account to our NewDot product following the setup details in this article.

@equationalapplications - Chronos is a 'contact' option in the NewDot chat. To start a conversation with Chronos, click the green + icon > New Chat > type Chronos.

2021-09-10_10-36-20 (1)

@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Sep 13, 2021

Doubled price to $500 - still waiting on proposals.

@pROFESOR11
Copy link
Contributor

@Christinadobrzyn Thanks for your reply. Well, I want to work the issue, but I am not sure what I am doing wrong, for some reason I can't reproduce the issue. I can't find Chronos in contacts. I tried to manually add chronos@expensify.com and wrote "Hey there!", but got no response. Should I write something specific to get an answer as in the issue description?

Screen.Recording.2021-09-13.at.11.52.54.mov

@parasharrajat
Copy link
Member

This issue is seemed to be fixed now. Please re-test @isagoico .
Screenshot_1631541715

@equationalapplications
Copy link

Hi @pROFESOR11! Sorry for the delayed response, Expensifail is our test account. You can create a test account to our NewDot product following the setup details in this article.

@equationalapplications - Chronos is a 'contact' option in the NewDot chat. To start a conversation with Chronos, click the green + icon > New Chat > type Chronos.

2021-09-10_10-36-20 (1)

@Christinadobrzyn thank you. I too cannot find Chronos after starting a new chat and searching for Chronos.

@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Sep 14, 2021

Thanks for confirming @equationalapplications!

Based on what @parasharrajat mentioned above, we'll have @isagoico or @kavimuru test this to see if it's fixed.

@parasharrajat
Copy link
Member

@isagoico Please retest.

@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Sep 16, 2021

Pinged Isabella in Slack

@isagoico
Copy link

We were able to reproduce the issue in latest build

Screen_Recording_20210916-130619_New.Expensify.mp4

@parasharrajat
Copy link
Member

parasharrajat commented Sep 16, 2021

Good Thanks @isagoico, I will look closely. Maybe come up with follow-up questions.

@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Sep 20, 2021

Looks like we're still waiting on proposals for this!

@equationalapplications
Copy link

I am not able to find Chronos in the chat search still running on my localhost

screenshot.mp4

.

@equationalapplications
Copy link

equationalapplications commented Sep 20, 2021

Also, I cannot add a test account from the development web server on localhost. The avatar for the test email appears, but I cannot click it.

Screenshot2.mp4

.

@equationalapplications
Copy link

Also, I cannot find Chronos in new.expensify.com in the browser.
Screenshot_20210920-115948

@parasharrajat
Copy link
Member

Finally, the solution is found.

Proposal

  1. Li items are extending beyond the parent width and trying flexShrink:1 which was suggested from Lib was not working as the container was allowing unrestricted width.

To fix this we have to restrict the container's width and then respectively ul tags.

We have to add styles for ul & ol in

tagStyles: {

      ul: {
            maxWidth: '100%',
            flex: 1, //Optional not needed but seems useful to me.
        },

Then we apply maxWidth: '100%' to

chatItemMessage: {

which fixes this issue.

BEFORE AFTER
issue fix

@johnmlee101
Copy link
Contributor

Looks good to me @parasharrajat !

@MelvinBot MelvinBot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 20, 2021
@johnmlee101
Copy link
Contributor

cc @Christinadobrzyn

@Christinadobrzyn
Copy link
Contributor

Awesome! Hired @parasharrajat in Upwork.

@Christinadobrzyn
Copy link
Contributor

still on hold for n6

@kadiealexander
Copy link
Contributor

Please refer to this post for updated information on the n6 hold, we've raised the bonus to $250 for all issues where a PR is created before the N6 hold is lifted.

@Christinadobrzyn
Copy link
Contributor

Not overdue - on hold for N6

@MelvinBot MelvinBot removed the Overdue label Oct 11, 2021
@Christinadobrzyn
Copy link
Contributor

On hold for n6

@Christinadobrzyn
Copy link
Contributor

N6 hold is removed as the newsletter has deployed.

Here's the PR for this job - a $250 bonus will also apply to this job!

@Christinadobrzyn
Copy link
Contributor

PR appears to be in staging - not overdue - waiting

@MelvinBot MelvinBot removed the Overdue label Oct 28, 2021
@mountiny mountiny changed the title Chat - Chronos message is cut off at the edge and not fully visible [HOLD for payment 2021-11-04] Chat - Chronos message is cut off at the edge and not fully visible Oct 30, 2021
@mountiny mountiny added the Awaiting Payment Auto-added when associated PR is deployed to production label Oct 30, 2021
@Christinadobrzyn
Copy link
Contributor

7-day production window is upon us. Paid @parasharrajat in Upwork and included a $250 bonus. Closing this. 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
None yet
Development

No branches or pull requests