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 - VBA - Unable to see dates clearly or missing altogether in company /personal info page #6438

Closed
kavimuru opened this issue Nov 23, 2021 · 58 comments
Assignees
Labels
Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Improvement Item broken or needs improvement.

Comments

@kavimuru
Copy link

kavimuru commented Nov 23, 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. Open app and login with an applause.expensifail account
  2. Select account workspace
  3. Navigate to “connect bank account”
  4. Log in with chase credentials
  5. Proceed to company info page
  6. Navigate to incorporation date and select a random date
  7. Add all required details and select continue
  8. Navigate to Date of birth in personal info page

Expected Result:

Months, year and day would be listed clearly in drop down or missing altogether

Actual Result:

For incorporation date - Only month displayed is December, displayed day starts from 24 and displayed year starts from 2022
For Date Of Birth - Dates are completely missing

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • iOS

Version Number: 1.1.16-1

Reproducible in staging?: Yes

Reproducible in production?: Yes

Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Bug5342304_IMG_1735.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:

View all open jobs on GitHub

@kavimuru kavimuru added the DeployBlockerCash This issue or pull request should block deployment label Nov 23, 2021
@OSBotify
Copy link
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.

@MelvinBot
Copy link

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

@marcaaron
Copy link
Contributor

Two potential candidates that may have caused this regression since they deal with the date picker:

#5939
#6192

If I had to guess I'd say this one is more likely the cause since it's adding a maximumDate attribute and I have no idea what the expected behavior for that should be:

https://github.com/Expensify/App/pull/6192/files

@marcaaron
Copy link
Contributor

marcaaron commented Nov 23, 2021

I think it is this package @react-native-community/datetimepicker but then just shoved inside a Popover component here:

<Popover
isVisible={this.state.isPickerVisible}
onClose={this.selectDate}
>
<View style={[
styles.flexRow,
styles.justifyContentBetween,
styles.borderBottom,
styles.pb1,
styles.ph4,
]}
>
<Button
title={this.props.translate('common.reset')}
color={themeColors.textError}
onPress={this.reset}
/>
<Button
title={this.props.translate('common.done')}
color={themeColors.link}
onPress={this.selectDate}
/>
</View>
<RNDatePicker
value={this.state.selectedDate}
mode="date"
display="spinner"
onChange={this.updateLocalDate}
locale={this.props.preferredLocale}
maximumDate={this.props.maximumDate}
/>

@marcaaron
Copy link
Contributor

Going to try to reproduce this now and see if reverting either of those PRs fixes the issue.

@sketchydroide
Copy link
Contributor

Sorry for a moment I thought this was the old app, and then I understood my misunderstanding and deleted the comment shamefully

@marcaaron
Copy link
Contributor

Haha all good thanks 😂

@marcaaron
Copy link
Contributor

Sorry. I'm not able to reproduce this at all so unsure how to continue investigating.

@marcaaron
Copy link
Contributor

marcaaron commented Nov 23, 2021

Need to update Xcode to test this - will look at the linked library issues - based on this comment most likely there is some issue with iOS 15 and nothing wrong with any new code that we have written.

@chiragsalian chiragsalian added Daily KSv2 Improvement Item broken or needs improvement. and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels Nov 23, 2021
@chiragsalian
Copy link
Contributor

Looks like the issue is reproducible on production so i'm removing the DeployBlockerCash label. Gonna still keep it as daily since its important to address soon.

@kidroca
Copy link
Contributor

kidroca commented Nov 24, 2021

Seems very similar to this: react-native-datetimepicker/datetimepicker#526 and this react-native-datetimepicker/datetimepicker#524

It's likely a problem with the library itself
Since we have DatePicker/index.ios.js maybe we can switch to a different library to address this faster

@marcaaron marcaaron removed their assignment Nov 24, 2021
@MelvinBot
Copy link

Huh... This is 4 days overdue. Who can take care of this?

@marcaaron
Copy link
Contributor

Another short term option would be to switch back to a regular text input on iOS 15 until this has been resolved.

@marcaaron
Copy link
Contributor

I'm not able to repro this on iOS sim running iOS 15. Going to drop this down to a weekly. We can bump it back up if anyone is able to repro locally and wants to fix.

@marcaaron marcaaron added Weekly KSv2 and removed Daily KSv2 labels Dec 1, 2021
@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Jun 28, 2022
@melvin-bot
Copy link

melvin-bot bot commented Jun 28, 2022

📣 @JosueEchandiaAsto You have been assigned to this job by @Beamanator!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@Beamanator
Copy link
Contributor

Hiring @JosueEchandiaAsto as I also like their proposal 👍

@mallenexpensify yes I think we should compensate $250 for reproduction steps & $250 for the fix

@Beamanator Beamanator changed the title iOS - VBA - Unable to see dates clearly or missing altogether in company /personal info page [$250] iOS - VBA - Unable to see dates clearly or missing altogether in company /personal info page Jun 28, 2022
@JosueEchandiaAsto
Copy link
Contributor

PR Created 🎉 #9599

@mallenexpensify mallenexpensify removed their assignment Jul 5, 2022
@mallenexpensify mallenexpensify added External Added to denote the issue can be worked on by a contributor and removed External Added to denote the issue can be worked on by a contributor labels Jul 5, 2022
@melvin-bot
Copy link

melvin-bot bot commented Jul 5, 2022

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

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Jul 5, 2022
@Beamanator
Copy link
Contributor

PR merged a week ago, still waiting for deployment. Changing priority back to Weekly

@Beamanator Beamanator added Weekly KSv2 and removed Daily KSv2 labels Jul 6, 2022
@laurenreidexpensify
Copy link
Contributor

Merged and on staging, should hit prod this week

@melvin-bot melvin-bot bot added the Overdue label Jul 19, 2022
@laurenreidexpensify
Copy link
Contributor

heh this hit prod 7 days ago. I'll issue payment tomorrow

@melvin-bot melvin-bot bot removed the Overdue label Jul 19, 2022
@laurenreidexpensify laurenreidexpensify added Daily KSv2 and removed Weekly KSv2 labels Jul 19, 2022
@laurenreidexpensify
Copy link
Contributor

@JosueEchandiaAsto @parasharrajat have just sent offers in Upwork - sorry for the delay! As soon as those are accepted I'll issue payment and close this out. Sorry that this wasn't the smoothest - the automation trigger failed here and that slowed us down from closing this out!

@laurenreidexpensify
Copy link
Contributor

all paid 👍🏽

@JosueEchandiaAsto
Copy link
Contributor

JosueEchandiaAsto commented Jul 20, 2022

Hi @laurenreidexpensify The compensation for the reproduction steps is pending 🥲
#6438 (comment)

@JosueEchandiaAsto
Copy link
Contributor

JosueEchandiaAsto commented Sep 15, 2022

some news?.. @laurenreidexpensify

CC: @mallenexpensify @Beamanator

@parasharrajat
Copy link
Member

Bump @mallenexpensify Maybe you can help @JosueEchandiaAsto here.

@mallenexpensify
Copy link
Contributor

Thanks for the ping @parasharrajat , for some reason I didn't see the post from you two days ago @JosueEchandiaAsto . Apologies for the delay in payment. I just issued you a $250 bonus on the same Upwork job. Thanks for the help!

@JosueEchandiaAsto
Copy link
Contributor

Thanks @mallenexpensify

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Improvement Item broken or needs improvement.
Projects
None yet
Development

No branches or pull requests