Closed lanitochka17 closed 2 months ago
:wave: 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:
Triggered auto assignment to @tgolen (DeployBlockerCash
), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.
@tgolen FYI 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
We think that this bug might be related to #vip-vsp
Android - Workspaces - "Workspaces" title is shown in two lines
When the screen is small we don't render the button outside of HeaderWithBackButton
.
We need to follow other workspace pages and render the button outside the HeaderWithBackButton
so it will be below the header.
<HeaderWithBackButton
title={translate('common.workspaces')}
shouldShowBackButton={isSmallScreenWidth}
onBackButtonPress={() => Navigation.goBack()}
icon={Illustrations.BigRocket}
>
{!isSmallScreenWidth && (
<Button
accessibilityLabel={translate('workspace.new.newWorkspace')}
success
medium
text={translate('workspace.new.newWorkspace')}
onPress={() => interceptAnonymousUser(() => App.createWorkspaceWithPolicyDraftAndNavigateToIt())}
/>
)}
</HeaderWithBackButton>
{isSmallScreenWidth && (
<View style={[styles.pl5, styles.pr5]}>
{' '}
<Button
accessibilityLabel={translate('workspace.new.newWorkspace')}
success
medium
text={translate('workspace.new.newWorkspace')}
onPress={() => interceptAnonymousUser(() => App.createWorkspaceWithPolicyDraftAndNavigateToIt())}
style={isSmallScreenWidth && styles.flexGrow1, isSmallScreenWidth && styles.mb3}
/>
</View>
)}
Note we also need to do the same when rendering the empty workspace section and we can create a function to return the button like we do in other components. https://github.com/Expensify/App/blob/425804aa005c6299ff0f59ff596764c429e19abd/src/pages/workspace/WorkspaceMembersPage.tsx#L534-L545
https://github.com/Expensify/App/assets/85894871/cc1797e0-aec6-4f90-b7e9-2bd7df457729
Job added to Upwork: https://www.upwork.com/jobs/~01db4b9a25dfd3bca7
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eh2077 (External
)
Triggered auto assignment to @johncschuster (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.
Thanks for the proposal @Krishna2323
It looks good to me! Can you please do a PR?
๐ฃ @Krishna2323 ๐ An offer has been automatically sent to your Upwork account for the Contributor role ๐ Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer 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 ๐
@tgolen, PR will be ready in a hour.
@tgolen, the PR is almost ready. I just need to record for Android native. I'm facing build issues and have been trying to resolve them for the last hour. Will update as soon as it get resolved.
I was also having build issues, Deleting android/app/.cxx
and rebuilding worked for me @Krishna2323 you may want to try that else let know i'll give you the screenshot/Video for Android : )
We can remove this from the deploy blocker list. It is not really a blocker. @tgolen
Thanks for the help, @ishpaul777, but it didn't work for me. It would be great if you could record a video for Android native. I moved the project code to an external SSD to free up some space, maybe that's causing the error.
@tgolen, PR is ready for review, all recordings have been added. @ishpaul777, thanks for the help.
I agree this does not have to be a blocker given the preconditions of narrow screen and increased base font size
I was C+ on the PR. Please assign me.
Reviewing
label has been removed, please complete the "BugZero Checklist".
The solution for this issue has been :rocket: deployed to production :rocket: in version 9.0.3-7 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
If no regressions arise, payment will be issued on 2024-07-10. :confetti_ball:
For reference, here are some details about the assignees on this issue:
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
Do you agree ๐ or ๐ ?
Issue is ready for payment but no BZ is assigned. @MitchExpensify you are the lucky winner! Please verify the payment summary looks correct and complete the checklist. Thanks!
Payment summary:
Paid and contract ended @Krishna2323
Payment requested as per https://github.com/Expensify/App/issues/44210#issuecomment-2221736353
$250 approved for @parasharrajat
If you havenโt already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.0.1-0 Reproducible in staging?: Y Reproducible in production?: N If this was caught during regression testing, add the test name, ID and link from TestRail: N/A Issue reported by: Applause - Internal Team
Action Performed:
Precondition:
Expected Result:
"Workspaces" title will not be shown in two lines
Actual Result:
"Workspaces" title is shown in two lines
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @eh2077