-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 2024-06-06] [$250] Workspace - RBR is aligned to top of WS bar #41609
Comments
Triggered auto assignment to @isabelastisser ( |
@isabelastisser 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. |
ProposalPlease re-state the problem that we are trying to solve in this issue.Workspace - RBR is aligned to top of WS bar What is the root cause of that problem?Styles applied to icon container and the main App/src/pages/workspace/WorkspacesListRow.tsx Lines 145 to 158 in 387ab0e
What changes do you think we should make in order to solve the problem?The icon container should not have the Also we need to apply styles to icon container for alignment and spacing for narrow layouts. We can create a style object like we do with App/src/pages/workspace/WorkspacesListRow.tsx Line 147 in 387ab0e
App/src/pages/workspace/WorkspacesListRow.tsx Line 158 in 387ab0e
isNarrow && {flexDirection: 'column', justifyContent: 'flex-start', marginLeft: 16, marginTop: 10}, What alternative solutions did you explore? (Optional)Resultrbr_alignment_workspace_list_row.mp4 |
Proposal Updated
|
Proposal Updated
|
Job added to Upwork: https://www.upwork.com/jobs/~01e565fa0b66eb131c |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia ( |
@isabelastisser Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Here's a proposal for the Expensify project: Title: Unifying Front-end with React Native Migration Proposal: I understand that Expensify is seeking to unify its front-end across platforms using React Native. The GitHub issue outlines the requirements for this migration. My proposal focuses on addressing the key challenges and providing a technical explanation of the changes I will make. Objectives:
Technical Approach:
Deliverables:
Timeline:
Total estimated time: 17 days Please review my proposal, and if accepted, I will post it on Upwork and begin working on the project. I will then submit the code for review and merge into the Expensify codebase. |
📣 @Isaac017! 📣
|
Contributor details Title: Unifying Front-end with React Native Migration Proposal: I understand that Expensify is seeking to unify its front-end across platforms using React Native. The GitHub issue outlines the requirements for this migration. My proposal focuses on addressing the key challenges and providing a technical explanation of the changes I will make. Objectives:
Technical Approach:
Deliverables:
Timeline:
Total estimated time: 17 days Please review my proposal, and if accepted, I will post it on Upwork and begin working on the project. I will then submit the code for review and merge into the Expensify codebase. |
|
Contributor isaacochagwu001@expensify.com https://www.upwork.com/freelancers/~01ecb4ea889e2c3a93 Title: Unifying Front-end with React Native Migration Proposal: I understand that Expensify is seeking to unify its front-end across platforms using React Native. The GitHub issue outlines the requirements for this migration. My proposal focuses on addressing the key challenges and providing a technical explanation of the changes I will make. Objectives: Migrate the existing front-end to React Native, ensuring a seamless user experience across platforms. Implement a consistent design language and component library. Ensure compatibility with existing APIs and backend infrastructure. Conduct thorough testing on multiple platforms (iOS, Android, and Web). Technical Approach: Set up a new React Native project, integrating it with the existing Expensify codebase. Develop a custom component library, utilizing React Native elements and Expensify's design guidelines. Implement platform-specific optimizations for performance and user experience. Utilize React Native's built-in navigation and routing features for a cohesive user flow. Conduct thorough testing, including unit tests, integration tests, and UI tests. Deliverables: A detailed report outlining the migration process and technical decisions. A fork of the Expensify/App repository with the implemented changes. A pull request for merging the changes into the main codebase. Screenshots and confirmation of successful testing on all platforms. Timeline: Initial setup and planning: 2 days Component library development: 5 days Platform-specific optimizations: 3 days Testing and debugging: 5 days Documentation and reporting: 2 days Total estimated time: 17 days Please review my proposal, and if accepted, I will post it on Upwork and begin working on the project. I will then submit the code for review and merge into the Expensify codebase. |
@thesahindia, can you please provide an update? Thanks! |
@Krishna2323's proposal looks good to me! 🎀 👀 🎀 C+ reviewed |
Triggered auto assignment to @MonilBhavsar, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
@MonilBhavsar, please review the proposals. Thanks! |
Looks good to me |
📣 @Krishna2323 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
@thesahindia, PR ready for review. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.77-11 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-06-06. 🎊 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:
|
@thesahindia, please complete the checklist before payment is due. Thanks! |
It was caused by #40069. I have commented on the PR. We should have tested for the error case in that PR. I don't think we need a test case here. |
Payment summary: @thesahindia requires payment through NewDot Manual Requests - C+ review $250 |
All set! |
$250 approved for @thesahindia |
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: 1.4.70-1
Reproducible in staging?: y
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
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:
Precondition: Sign up new account.
Expected Result:
RBR is aligned to center of WS bar
Actual Result:
RBR is aligned to top of WS bar
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: @isabelastisserThe text was updated successfully, but these errors were encountered: