Closed mananjadhav closed 1 week ago
On Hold for https://github.com/Expensify/App/pull/41561. Waiting for ConnectionLayout
padding fix to get merged so that I can create the screencasts for the mapping screens.
@fedirjh Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]
@Expensify/design For your review.
@mananjadhav i can't see cost center and region
@rushatgabhane Did you check the pre-requisite in the Test steps? and did you try to select the Demo company
?
Pre-requisite:
- Xero account setup in an existing workspace.
- You need to have 2 tracking categories named "Region" and "Cost centers", to set their mapping.
- On your Xero account, select the Demo company organization.
- Go to Accounting > Advanced, select "Tracking categories".
- You should see something named "Region", in reality it can be any name that's why you must show/hide the option depending on the name of the tracking category.
- Add a new tracking category named "Cost centers".
@mananjadhav i can't see cost center and region
To follow up with @mananjadhav, please see this discussion for context on how to enable them.
@lakchote I didn't understand what you meant? The link also doesn't take me to any specific thread. I already added the pre-requisite on how to add them based on your instructions in slack. Is there anything else that's expected from me?
Design-wise this looks good to me.
This is because the category options are hardcoded to Tags and Default. But I selected Report Field
Got it. We'll be adding this in future. But at the moment I think it should show blank here then? @trjExpensify @lakchote ?
### Fixed Issues
section aboveTests
sectionQA steps
sectiontoggleReport
and not onIconClick
).myBool && <MyComponent />
.src/languages/*
files and using the translation methodSTYLE.md
) were followedAvatar
, I verified the components using Avatar
have been tested & I retested again)/** comment above it */
this
properly so there are no scoping issues (i.e. for onClick={this.submit}
the method this.submit
should be bound to this
in the constructor)this
are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this);
if this.submit
is never passed to a component event handler like onClick
)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG
)Avatar
is modified, I verified that Avatar
is working as expected in all cases)Design
label and/or tagged @Expensify/design
so the design team can review the changes.ScrollView
component to make it scrollable when more elements are added to the page.main
branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test
steps.@lakchote I didn't understand what you meant? The link also doesn't take me to any specific thread. I already added the pre-requisite on how to add them based on your instructions in slack. Is there anything else that's expected from me?
It was meant for @rushatgabhane to help him enable mapping options.
This is because the category options are hardcoded to Tags and Default. But I selected Report Field
Got it. We'll be adding this in future. But at the moment I think it should show blank here then? @trjExpensify @lakchote ?
Report field will be added once we'll port Control
workspaces to NewDot. It's a Control feature.
I'd move in favour of showing blank there in the meantime.
Thanks for the comment @lakchote. I'll push an update in a while for the Report field.
@rushatgabhane @lakchote Added a fix to show blank for the report field. I didn't refactor too much because we are anyway going remove it later.
:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.
I have merged by mistake (since we're on merge freeze). Since it's behind beta, and it should not influence any user behaviour we don't need to revert it.
I've created a new xero-merge-freeze
branch for the next Xero PRs, people will need to use it as a base instead of main
in the meantime to not block progress.
π Deployed to staging by https://github.com/lakchote in version: 1.4.72-0 π
platform | result |
---|---|
π€ android π€ | success β |
π₯ desktop π₯ | success β |
π iOS π | success β |
πΈ web πΈ | success β |
Tell us about PR Pre-props: step 4 cannot be completed because we do not see "Tracking categories" Go to Accounting > Advanced, select Tracking Categories. How do we set up an account to complete these steps? thank you in advance
https://github.com/Expensify/App/assets/78819774/4d56ce08-77d3-4a7d-b3fb-1de2ec19d8b4
@lanitochka17 Did you try the steps mentioned in the pre-requisites? Please find attached video on navigating through Xero. You should see Demo Company when you create a new account on Xero, and use that for syncing when you test it.
https://github.com/Expensify/App/assets/3069065/48094376-10db-4e18-b4fa-586d0a4ee134
@mananjadhav Thanks for sharing the video again. I have a question, How to verify the PR step 6 and 8.d and e? Is this can only be done internally?
You should be able to view this in the application data onyxdb in Chrom Dev tools. I've added a note in the steps along with the screenshot.
π Deployed to production by https://github.com/Beamanator in version: 1.4.72-1 π
platform | result |
---|---|
π€ android π€ | success β |
π₯ desktop π₯ | success β |
π iOS π | success β |
πΈ web πΈ | success β |
Details
Fixed Issues
$ https://github.com/Expensify/App/issues/39732 PROPOSAL:
Tests
Pre-requisite:
https://github.com/Expensify/App/assets/3069065/48094376-10db-4e18-b4fa-586d0a4ee134
Steps
Tracking Categories
Import
toggle switch.Import
to On, in the data you can verify thatpolicy_<policyID>.connections.xero.config.importTrackingCategories
is set to true. On switching off, it is set to false.Map Xero cost centers to
andMap Xero regions to
.Map Xero <fieldname> to
. b. You should see two optionsXero contact default
andTags
as two menu items. c. On selecting any one of the options, you should be redirected back to the tracking categories page, and the selected option should show up against each field title. d. Inpolicy_<policyID>.connections.xero.mappings.trackingCategory_<trackingCategoryId>
you should see the selected value , ie,DEFAULT
forXero contact default
andTAG
forTags
. e. The trackingCategoryId should be based on theid
field ofpolicy_<policyID>.connections.xero.trackingCategories.name
matched to cost centers and regions respectively.For all the data verification related info, go to Chrom Dev Tools -> Application -> Storage -> OnyxDB -> Search for your policy_. Check out the data in the connections object as shown in the screenshot below.
Offline tests
Same as Tests.
QA Steps
Same as Tests.
PR Author Checklist
### Fixed Issues
section aboveTests
sectionOffline steps
sectionQA steps
sectiontoggleReport
and notonIconClick
)myBool && <MyComponent />
.src/languages/*
files and using the translation methodSTYLE.md
) were followedAvatar
, I verified the components usingAvatar
are working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)
)Avatar
is modified, I verified thatAvatar
is working as expected in all cases)Design
label and/or tagged@Expensify/design
so the design team can review the changes.ScrollView
component to make it scrollable when more elements are added to the page.main
branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTest
steps.Screenshots/Videos
Android: Native
https://github.com/Expensify/App/assets/3069065/bc492e12-81d7-4662-ab37-602b690fd9d5Android: mWeb Chrome
https://github.com/Expensify/App/assets/3069065/a34a23fd-fd59-48cb-8fb3-28e257b015bfiOS: Native
https://github.com/Expensify/App/assets/3069065/104b5d0b-bda5-449e-adaa-57f088a07e44iOS: mWeb Safari
https://github.com/Expensify/App/assets/3069065/51a1aa66-97e3-4803-ba41-620924d35b6dMacOS: Chrome / Safari
https://github.com/Expensify/App/assets/3069065/ea3dd2f9-3147-4921-b0ff-d748c80bb12eMacOS: Desktop
https://github.com/Expensify/App/assets/3069065/28197ed0-3d81-40d0-ace1-12e36ba1caac