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 2023-01-31] IOS - New Room - Unable create a room when using second dash #14137

Closed
1 task done
kbecciv opened this issue Jan 9, 2023 · 16 comments
Closed
1 task done
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review

Comments

@kbecciv
Copy link

kbecciv commented Jan 9, 2023

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


Issue found when executing PR #13994

Action Performed:

  1. Launch the app
  2. Log i with expensifail account
  3. Tap on Fub menu
  4. Select New Room
  5. Use two dashes in the room name field

Expected Result:

Able to use dashes when creating a room

Actual Result:

Unable create a room when using second dash

Workaround:

Unknown

Platforms:

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

  • iOS / native

Version Number: 1.2.50.13

Reproducible in staging?: Yes

Reproducible in production?: n/a

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

RPReplay_Final1673192007.1.MP4
RPReplay_Final1673278692.MP4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01eefd02d7b3fd0024
  • Upwork Job ID: 1615503215369134080
  • Last Price Increase: 2023-01-18
@kbecciv kbecciv added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jan 9, 2023
@melvin-bot melvin-bot bot locked and limited conversation to collaborators Jan 9, 2023
@melvin-bot melvin-bot bot added the Overdue label Jan 11, 2023
@maddylewis
Copy link
Contributor

interesting! so, if a room can be made with a single hyphen, a room should be made with a double hyphen. therefore, we should build out the capability to create a room with a double hyphen.

this doesn't seem like it would affect the standard end-user experience too much, but the error messaging + capability should be consistent.

i will work on reproducing and then go from there!

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jan 11, 2023
@maddylewis

This comment was marked as off-topic.

@melvin-bot melvin-bot bot removed the Overdue label Jan 17, 2023
@kbecciv
Copy link
Author

kbecciv commented Jan 17, 2023

@maddylewis This is IOS app issue only.

@maddylewis
Copy link
Contributor

oh got it - thanks for clarifying. let me try to reproduce on the app.

@luacmartins
Copy link
Contributor

luacmartins commented Jan 18, 2023

I can reproduce on iOS. I think the issue is that iOS combines -- into which is rejected by our validation. I can work on a fix.

@luacmartins luacmartins self-assigned this Jan 18, 2023
@luacmartins luacmartins added the Internal Requires API changes or must be handled by Expensify staff label Jan 18, 2023
@melvin-bot melvin-bot bot unlocked this conversation Jan 18, 2023
@melvin-bot
Copy link

melvin-bot bot commented Jan 18, 2023

Job added to Upwork: https://www.upwork.com/jobs/~01eefd02d7b3fd0024

@melvin-bot
Copy link

melvin-bot bot commented Jan 18, 2023

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

@melvin-bot melvin-bot bot added the Reviewing Has a PR in review label Jan 19, 2023
@luacmartins
Copy link
Contributor

PR up

@luacmartins
Copy link
Contributor

PR in prod!

@luacmartins luacmartins changed the title IOS - New Room - Unable create a room when using second dash [HOLD for payment 2023-01-31] IOS - New Room - Unable create a room when using second dash Jan 25, 2023
@mollfpr
Copy link
Contributor

mollfpr commented Feb 1, 2023

@maddylewis Could you help issue the payment here? Thanks!

@maddylewis
Copy link
Contributor

@mollfpr - will you apply to the job so that i can issue payment? https://www.upwork.com/jobs/~01eefd02d7b3fd0024

thanks!

@mollfpr
Copy link
Contributor

mollfpr commented Feb 1, 2023

@maddylewis applied thanks!

@maddylewis
Copy link
Contributor

alrighty, i think we're all paid up on this one.

@mollfpr
Copy link
Contributor

mollfpr commented Feb 1, 2023

@maddylewis offer accepted!

@luacmartins
Copy link
Contributor

@maddylewis have we handled all payments for this issue? Can we close this?

@dylanexpensify
Copy link
Contributor

Payment sent!!

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. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review
Projects
None yet
Development

No branches or pull requests

5 participants