Skip to content

[HOLD for payment 2023-10-02] [HOLD for payment 2023-09-29] [$500] Dev: Web - App crashing on creating new workspace #27782

Closed

Description

@kbecciv

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. Login with new account
  2. Click on FAB > New workspace
  3. App crashes

Expected Result:

App should not crash and workspace is created

Actual Result:

App crashes and a console error is present

Workaround:

Unknown

Platforms:

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

  • Android / native
  • Android / Chrome
  • iOS / native
  • iOS / Safari
  • MacOS / Chrome / Safari
  • MacOS / Desktop

Version Number: Dev 1.3.71-6
Reproducible in staging?: n
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
Notes/Photos/Videos: Any additional supporting documentation

app-crashes.mp4

Expensify/Expensify Issue URL:
Issue reported by: @aman-atg
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695116344613649

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~016db5c84825401ac4
  • Upwork Job ID: 1704127946717769728
  • Last Price Increase: 2023-09-19

Activity

  1. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Sep 19, 2023
  2. changed the title [-]Dev: Web - App crashing on creating new workspace[/-] [+][$500] Dev: Web - App crashing on creating new workspace[/+] on Sep 19, 2023
  3. melvin-bot commented on Sep 19, 2023

    @melvin-bot
  4. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Sep 19, 2023
  5. melvin-bot commented on Sep 19, 2023

    @melvin-bot

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

  6. melvin-bot commented on Sep 19, 2023

    @melvin-bot

    Triggered auto assignment to Contributor-plus team member for initial proposal review - @ArekChr (External)

  7. jeet-dhandha commented on Sep 19, 2023

    @jeet-dhandha
    Contributor

    Proposal

    Please re-state the problem that we are trying to solve in this issue.

    • Web app crashing when creating new workspace from FAB.

    What is the root cause of that problem?

    • hideReactionList is not defined as its called before the ref could by the parent ref before the innerRef was set.
    Screenshot 2023-09-19 at 7 20 41 PM

    What changes do you think we should make in order to solve the problem?

    • We can add checks to see if ref and hideReactionList is defined before calling it.

    const hideReactionList = () => {
    innerReactionListRef.current.hideReactionList();
    };

    if (!innerReactionListRef.current || !innerReactionListRef.current.hideReactionList) {
        return;
    }

    What alternative solutions did you explore? (Optional)

  8. Pujan92 commented on Sep 19, 2023

    @Pujan92
    Contributor

    Seems a regression from #27268 as for me while navigating back from profile crashes the app
    Screenshot 2023-09-19 at 20 46 28

  9. PiyushChandra17 commented on Sep 19, 2023

    @PiyushChandra17
    Contributor

    Proposal

    Please re-state the problem that we are trying to solve in this issue.

    Web - App crashing on creating new workspace

    What is the root cause of that problem?

    hideReactionList is not defined, need to check initialization first

    What changes do you think we should make in order to solve the problem?

    We need to check this condition first:
    if (innerReactionListRef.current && innerReactionListRef.current.hideReactionList) {

    here,

    const hideReactionList = () => {
    innerReactionListRef.current.hideReactionList();
    };

    if (innerReactionListRef.current && innerReactionListRef.current.hideReactionList) {
                innerReactionListRef.current.hideReactionList();
            }
    
    

    What alternative solutions did you explore? (Optional)

    NA

  10. 40 remaining items

  11. melvin-bot commented on Oct 4, 2023

    @melvin-bot

    @ArekChr, @mountiny, @muttmuure Huh... This is 4 days overdue. Who can take care of this?

  12. mountiny commented on Oct 4, 2023

    @mountiny
    Contributor

    @muttmuure Can you please handle the payment of $50 here and then we can close

  13. muttmuure commented on Oct 5, 2023

    @muttmuure
    Contributor

    Invite sent to @aman-atg

  14. aman-atg commented on Oct 5, 2023

    @aman-atg
    Contributor

    @muttmuure Submitted the proposal on Upwork

  15. melvin-bot commented on Oct 10, 2023

    @melvin-bot

    @ArekChr, @mountiny, @muttmuure Huh... This is 4 days overdue. Who can take care of this?

  16. muttmuure commented on Oct 11, 2023

    @muttmuure
    Contributor

    Handling

  17. muttmuure commented on Oct 11, 2023

    @muttmuure
    Contributor

    Offer sent

  18. melvin-bot commented on Oct 17, 2023

    @melvin-bot

    @ArekChr, @mountiny, @muttmuure Huh... This is 4 days overdue. Who can take care of this?

  19. muttmuure commented on Oct 17, 2023

    @muttmuure
    Contributor

    Paid!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionDailyKSv2EngineeringExternalAdded to denote the issue can be worked on by a contributor

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions