Skip to content

Join request - Accept and Decline button from Join request does not work in thread #37964

Closed

Description

@izarutskaya

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


Found when validating PR : #37525

Version Number: 1.4.49-0
Reproducible in staging?: Y
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team

Action Performed:

Precondition:

  1. Go to staging.new.expensify.com as non-member.
  2. Go to #admin of the workspace from precondition.
  3. Right click on join request message > Reply in thread,
  4. Click Accept or Decline.

Expected Result:

Accept and Decline button works in thread.

Actual Result:

Accept and Decline button does not work in thread.

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

Bug6406752_1709890976617.20240308_173102.mp4

View all open jobs on GitHub

Activity

  1. added
    DeployBlockerCashThis issue or pull request should block deployment
    BugSomething is broken. Auto assigns a BugZero manager.
    on Mar 8, 2024
  2. melvin-bot commented on Mar 8, 2024

    @melvin-bot

    Triggered auto assignment to @lschurr (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

  3. github-actions commented on Mar 8, 2024

    @github-actions
    Contributor

    👋 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:

    1. Identify the pull request that introduced this issue and revert it.
    2. Find someone who can quickly fix the issue.
    3. Fix the issue yourself.
  4. melvin-bot commented on Mar 8, 2024

    @melvin-bot

    Triggered auto assignment to @rlinoz (Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

  5. izarutskaya commented on Mar 8, 2024

    @izarutskaya
    Author

    @rlinoz 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.

  6. izarutskaya commented on Mar 8, 2024

    @izarutskaya
    Author

    We think that this bug might be related to #wave8-collect-admins
    CC @zanyrenney

  7. narefyev91 commented on Mar 8, 2024

    @narefyev91
    Contributor

    Will pick this one - in case it related to Invite member PR - i think we need to disable ability to make a thread for Accept/Reject requests. Same as we have for whisper option

  8. rlinoz commented on Mar 8, 2024

    @rlinoz
    Contributor

    Hmm I can't reproduce the issue, it takes a while to get accepted, but it works.

    Screen.Recording.2024-03-08.at.09.13.29.mov
  9. rlinoz commented on Mar 8, 2024

    @rlinoz
    Contributor

    i think we need to disable ability to make a thread for Accept/Reject requests.

    I think this is a different case where it is important to allow admins to start a thread out of a request to join the workspace

  10. narefyev91 commented on Mar 8, 2024

    @narefyev91
    Contributor

    Works for me on latest main

    Screen.Recording.2024-03-08.at.14.18.38.mov
  11. rlinoz commented on Mar 8, 2024

    @rlinoz
    Contributor

    Yeah, I am testing in staging and it works as well

    Screen.Recording.2024-03-08.at.09.19.11.mov
  12. rlinoz commented on Mar 8, 2024

    @rlinoz
    Contributor

    @narefyev91 correct me if I am wrong, but there is also no way to create the invite link in product right now, right?

  13. narefyev91 commented on Mar 8, 2024

    @narefyev91
    Contributor

    @narefyev91 correct me if I am wrong, but there is also no way to create the invite link in product right now, right?

    Yup you are right

  14. rlinoz commented on Mar 8, 2024

    @rlinoz
    Contributor

    Great, that makes this a non blocker

  15. added and removed
    DeployBlockerCashThis issue or pull request should block deployment
    on Mar 8, 2024
  16. rlinoz commented on Mar 11, 2024

    @rlinoz
    Contributor

    Tested again today, still can't reproduce, so closing this one.

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

Metadata

Metadata

Assignees

Labels

BugSomething is broken. Auto assigns a BugZero manager.DailyKSv2Engineering

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions