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

[pull] main from Expensify:main #44

Merged
merged 367 commits into from
Nov 3, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
367 commits
Select commit Hold shift + click to select a range
28e90a2
Merge remote-tracking branch 'upstream/main' into feat-19810-Replace-…
waterim Oct 27, 2023
ba0855c
fixes
waterim Oct 27, 2023
0409b54
fix: Copy to clipboard paid IOU in IOU page, instead of paid text req…
dukenv0307 Oct 27, 2023
398ec7f
Move QRCode component out of the folder
VickyStash Oct 27, 2023
87cb616
Merge branch 'main' into perf/report-screen-tti-2
kacper-mikolajczak Oct 27, 2023
64ae232
Merge branch 'main' into ts-migration/FullscreenLoadingIndicator
MaciejSWM Oct 27, 2023
8661910
Remove a few managerEmail uses
Beamanator Oct 27, 2023
b98acd7
Remove a few more managerEmails
Beamanator Oct 27, 2023
f90fcd4
export function to be used elsewhere
Beamanator Oct 27, 2023
a5fd715
fallback to hidden for report action item header
lukemorawski Oct 27, 2023
aa9637a
fallback to hidden for report welcome text
lukemorawski Oct 27, 2023
dc45365
Fix test
Beamanator Oct 27, 2023
bd28c35
ermagerd
Beamanator Oct 27, 2023
3dd8321
Merge branch 'main' of github.com:Expensify/App into beaman-removeRep…
Beamanator Oct 27, 2023
64cb597
Update and rename File-A-Dispute.md to Dispute-A-Transaction.md
joekaufmanexpensify Oct 27, 2023
73d685d
Resolve merge conflicts
BartoszGrajdek Oct 29, 2023
9074f0b
Fix lint problems for OpacityView
BartoszGrajdek Oct 29, 2023
bf8bec8
Merge branch 'main' into fix/28925-public-room-anonymous-user-able-to…
DylanDylann Oct 30, 2023
d7fe00b
fix: resolve conflict
HezekielT Oct 24, 2023
56b8969
remove unnecessary change
HezekielT Oct 24, 2023
864314d
fix lint issue
HezekielT Oct 24, 2023
58b8541
fix unused variables
HezekielT Oct 24, 2023
0e85cc5
Rename updateScaleRef
HezekielT Oct 25, 2023
73d1d0d
fix: resolve conflict
HezekielT Oct 30, 2023
1cb4cc1
fix: lint
DylanDylann Oct 30, 2023
d0ca77b
fix: resolve conflict
HezekielT Oct 30, 2023
3ff9451
fix lint prettier
DylanDylann Oct 30, 2023
8fd691c
run prettier
HezekielT Oct 30, 2023
215cc1a
fix run prettier
HezekielT Oct 30, 2023
310fead
fix addressed another prettier issue
HezekielT Oct 30, 2023
67a31dc
fix prettier
HezekielT Oct 30, 2023
1ce8ffd
Merge branch 'main' into ts-migration/SafeAreaConsumer
blazejkustra Oct 30, 2023
e9d0ab2
Merge branch 'main' of github.com:kubabutkiewicz/expensify-app into t…
kubabutkiewicz Oct 30, 2023
c15d77b
fix: lint
kubabutkiewicz Oct 30, 2023
1945ce6
add missing function execution
kacper-mikolajczak Oct 30, 2023
b88c47a
Merge branch 'Expensify:main' into fix--Android-pdf-scrolling-issue
HezekielT Oct 30, 2023
bd84424
Merge branch 'main' into perf/report-screen-tti-2
kacper-mikolajczak Oct 30, 2023
b4a9f5d
resolve aliases in CurrentUserPersonalDetails
kacper-mikolajczak Oct 30, 2023
e33e0d9
Merge branch 'Expensify:main' into reassure/get-last-visible-action
adhorodyski Oct 30, 2023
1b9c80f
fix prettier issue
HezekielT Oct 30, 2023
ba8d66f
Merge branch 'main' of https://github.com/hurali97/Expensify-App into…
hurali97 Oct 30, 2023
1fb0e5b
chore: satisfy linter, remove redundnat double underscore placeholder
adhorodyski Oct 30, 2023
a10f8b4
remove unnecessary screenshot/videos section from issue template
Julesssss Oct 30, 2023
201e1be
Merge branch 'main' into @swm/limit-report-routes-number
adamgrzybowski Oct 30, 2023
2a77766
fix prettier issues
kacper-mikolajczak Oct 30, 2023
7e8283d
Merge remote-tracking branch 'origin/main' into ts/component/SVGImage
fabioh8010 Oct 30, 2023
e40f602
Merge branch 'Expensify:main' into fix--Android-pdf-scrolling-issue
HezekielT Oct 30, 2023
892ea85
fix lint issues
HezekielT Oct 30, 2023
7b5ec6b
call updateScale in onScaleChanged instead of passing it as a prop
HezekielT Oct 30, 2023
f8b184a
chore: apply code review updates for naming unused parameters
adhorodyski Oct 30, 2023
480e84e
create optimistic personal detail only if it doesn't exist
bernhardoj Oct 30, 2023
02f3f9e
Merge remote-tracking branch 'upstream/main' into feat-19810-Replace-…
waterim Oct 30, 2023
918a2f4
fix merge
waterim Oct 30, 2023
1f8290f
Merge remote-tracking branch 'origin/main' into longPressForTaskPreviews
ShogunFire Oct 30, 2023
61da9cd
Merge was wrong
ShogunFire Oct 30, 2023
32c9065
lint fix
waterim Oct 30, 2023
3f03a24
Parsing the welcome message into html in create new room action
abzokhattab Oct 30, 2023
58cecfa
Improve the imports for lint
ShogunFire Oct 30, 2023
9b1e0e0
Merge branch 'main' into puneet-expensify-owner
puneetlath Oct 30, 2023
481c5df
abstract canCreateTaskInReport logic into function
puneetlath Oct 30, 2023
baea0d2
prettier
puneetlath Oct 30, 2023
6d8043a
After npm run prettier
ShogunFire Oct 30, 2023
df54e77
Merge branch 'main' into fix/28660-composer-height-not-resized
bernhardoj Oct 31, 2023
48aa3dc
add missing import
bernhardoj Oct 31, 2023
d66e573
fix: typecheck
hurali97 Oct 31, 2023
8827331
Merge branch 'main' of github.com:Expensify/App into beaman-removeRep…
Beamanator Oct 31, 2023
b9c613e
Merge branch 'main' into @swm/limit-report-routes-number
adamgrzybowski Oct 31, 2023
071bacb
Move sleep to beginning of each test
ospfranco Oct 31, 2023
0682682
Add cooldown between tests
ospfranco Oct 31, 2023
f8acc3c
Allow sleep between runs
ospfranco Oct 31, 2023
54dfef5
tooltip user name fallback to hidden
lukemorawski Oct 31, 2023
03edf32
Merge branch 'main' into 27393-displayName_fallback_to_hidden
lukemorawski Oct 31, 2023
d9559b8
merge main into branch
puneetlath Oct 31, 2023
47dede3
prettier
lukemorawski Oct 31, 2023
5da00d1
Modify runs and cooldowns
ospfranco Oct 31, 2023
0e14441
feat: initial setup for report screen test
koko57 Oct 11, 2023
299d80c
feat: add libs mocks
koko57 Oct 12, 2023
878fd0f
feat: mock other hooks
koko57 Oct 17, 2023
887b6e6
add perf tests to the ReportScreen
OlimpiaZurek Oct 26, 2023
d3c7128
changes after cr
OlimpiaZurek Oct 26, 2023
9a8800d
Remove ESLINT comment
ospfranco Oct 31, 2023
accfab9
lint fix
OlimpiaZurek Oct 31, 2023
0bf5289
Remove modified package script
ospfranco Oct 31, 2023
2af3d61
Add code comments
ospfranco Oct 31, 2023
1c233a3
fix clipboard in report utils
saranshbalyan-1234 Oct 31, 2023
b5c4c95
Update translation
hoangzinh Oct 31, 2023
22cfb1d
Merge branch 'main' into df/27508
hoangzinh Oct 31, 2023
41c05ae
Fix lint
hoangzinh Oct 31, 2023
274f686
Fix lint
hoangzinh Oct 31, 2023
c6adba2
Increate timeout to AWS DF action
ospfranco Oct 31, 2023
0fd6982
Merge branch 'main' into ts-migration/FullscreenLoadingIndicator
MaciejSWM Oct 31, 2023
8bc920f
Use isDM check instead of policy owner
puneetlath Oct 31, 2023
f35acb9
Add isExpensifyOnlyParticipantInReport function back since its used b…
puneetlath Oct 31, 2023
70a10f7
Reduce isPolicyOwnedByExpensifyAccounts to one line
puneetlath Oct 31, 2023
8c6acd7
fixes the issue
ishpaul777 Oct 31, 2023
d097f0e
added hook description
sarious Oct 20, 2023
6a989ec
apply shouldEnableMinHeight only for Manual tab. added property to pr…
sarious Oct 24, 2023
df084d5
fix the bug with screen width inside the edit money request flow
sarious Oct 24, 2023
b2af5ce
fix the bug with bottom inset, when buttons at the bottom are covered…
sarious Oct 24, 2023
7c2a515
enabled min height for all tabs
sarious Oct 30, 2023
d8c6af8
Merge commit '342a2c4a63f61c9129873f3f0d4b0616ea9e8083' into fix/1786…
sarious Oct 31, 2023
940d651
Update en.ts
Christinadobrzyn Oct 31, 2023
5f8ee37
Update Auto-Reconciliation.md
SofiedeVreese Oct 31, 2023
4e02b08
Update es.ts
Christinadobrzyn Oct 31, 2023
bbd8737
use aliases. get rid of useInitialWindowDimensions native file
sarious Oct 31, 2023
06879a6
Update translation key
hoangzinh Oct 31, 2023
ae9815c
fix eslint errors. added jsdoc returns line
sarious Oct 31, 2023
8dae3a0
fix: optimistically adding the new assignee's account id to a report'…
Tony-MK Nov 1, 2023
b00b220
fix app crash when creating task
DylanDylann Nov 1, 2023
1363569
Merge branch 'Expensify:main' into 30376-no-currency-for-tbd
rojiphil Nov 1, 2023
a63f39d
refactor: removing extra tab for prettier code
Tony-MK Nov 1, 2023
0afb9dd
removed unnecessary comment
rojiphil Nov 1, 2023
26c2752
Merge pull request #29140 from Expensify/SofiedeVreese-patch-2
nkuoch Nov 1, 2023
4a2e3eb
Update tests/e2e/testRunner.js
ospfranco Nov 1, 2023
b57460d
Merge pull request #30019 from Expensify/puneet-expensify-owner
grgia Nov 1, 2023
6a76a12
Merge pull request #29845 from bernhardoj/fix/28660-composer-height-n…
grgia Nov 1, 2023
4f1fd91
Merge pull request #30591 from bernhardoj/fix/30456-personal-details-…
grgia Nov 1, 2023
a6b0b54
Merge pull request #30635 from margelo/osp/e2e-improvements
mountiny Nov 1, 2023
e04afc6
Merge pull request #30414 from OlimpiaZurek/feat/27641/reassure-lhn-r…
mountiny Nov 1, 2023
22640ff
refetch wallet data
Pujan92 Nov 1, 2023
1a4a38b
Merge pull request #29984 from waterim/feat-19810-Replace-datepicker-2
mountiny Nov 1, 2023
fd89a41
Merge pull request #30263 from TMisiukiewicz/fix/laggy-search-opening
mountiny Nov 1, 2023
6cc68ac
Merge pull request #30482 from adhorodyski/reassure/get-last-visible-…
mountiny Nov 1, 2023
0f46976
Merge branch 'main' into jules-updateChecklist
Julesssss Nov 1, 2023
c8e7ee5
Merge pull request #30574 from Expensify/jules-updateChecklist
rlinoz Nov 1, 2023
20e21ab
Merge branch 'main' into fix/29767
Pujan92 Nov 1, 2023
be676e8
Merge branch 'main' into fix/29878
dukenv0307 Nov 1, 2023
233f97a
revert the condition
dukenv0307 Nov 1, 2023
60988f8
refactor: applying suggestion to make code look cleaner
Tony-MK Nov 1, 2023
6ac2746
fix Android inconsistent pixel of +n triangle corner
eh2077 Nov 1, 2023
347df0c
Merge pull request #30478 from dukenv0307/fix/29878
MonilBhavsar Nov 1, 2023
7d31ff4
fix lint
eh2077 Nov 1, 2023
b4e7978
Update docs/articles/expensify-classic/insights-and-custom-reporting/…
sonialiap Nov 1, 2023
4bf45db
Update Fringe-Benefits.md
sonialiap Nov 1, 2023
2a4b05a
Update docs/articles/expensify-classic/insights-and-custom-reporting/…
sonialiap Nov 1, 2023
bc496a8
Update docs/articles/expensify-classic/insights-and-custom-reporting/…
sonialiap Nov 1, 2023
a2d1f4f
Update docs/articles/expensify-classic/insights-and-custom-reporting/…
sonialiap Nov 1, 2023
b9e202d
Merge pull request #30342 from lukemorawski/27393-displayName_fallbac…
puneetlath Nov 1, 2023
d7e1461
Revert "[Form Provider Refactor] AddDebitCardPage"
Beamanator Nov 1, 2023
3488601
Oops missed a revert
Beamanator Nov 1, 2023
fca1c7d
Update footer.html
jamesdeanexpensify Nov 1, 2023
a036748
Delete docs/articles/expensify-classic/send-payments/Pay-Invoices.md
maddylewis Nov 1, 2023
c581cc2
Delete docs/articles/expensify-classic/expensify-card/Connect-To-Indi…
maddylewis Nov 1, 2023
9f7d6b7
Revert "27456 Chat - The green line is displayed chaotically at the r…
MonilBhavsar Nov 1, 2023
c864bdc
Delete docs/articles/expensify-classic/get-paid-back/Third-Party-Paym…
maddylewis Nov 1, 2023
a59867a
Delete docs/articles/expensify-classic/integrations/travel-integratio…
maddylewis Nov 1, 2023
a42804e
Delete docs/articles/expensify-classic/bank-accounts-and-credit-cards…
maddylewis Nov 1, 2023
0f21275
Merge pull request #30065 from sarious/fix/17866
luacmartins Nov 1, 2023
f175770
Update and rename Overview.md to Billing-Overview.md
maddylewis Nov 1, 2023
1ed6f89
Update and rename Card-Settings.md to Admin-Card-Settings-and-Feature…
maddylewis Nov 1, 2023
1921a93
Merge pull request #30708 from Expensify/beaman-revertFormRefactorAdd…
tgolen Nov 1, 2023
83a4cce
Update version to 1.3.94-1
OSBotify Nov 1, 2023
f9d11c5
Merge pull request #30671 from Tony-MK/fix/29228
luacmartins Nov 1, 2023
21931d0
Merge pull request #30709 from Expensify/jamesdeanexpensify-patch-1
NikkiWines Nov 1, 2023
aa18b6d
Merge pull request #30710 from Expensify/maddylewis-patch-2
marcochavezf Nov 1, 2023
bb51400
Merge pull request #30609 from abzokhattab/parsing-welcome-msg-in-cre…
neil-marcellini Nov 1, 2023
e108378
Merge pull request #30712 from Expensify/revert-29860-27456-green-line
Beamanator Nov 1, 2023
0319bf6
Merge pull request #30640 from saranshbalyan-1234/fix-cliboard-send-m…
jasperhuangg Nov 1, 2023
2723573
Default to true to make type happy
Beamanator Nov 1, 2023
254715f
wtf happened to you
Beamanator Nov 1, 2023
a4dd281
Merge pull request #30731 from Expensify/beaman-fixTypecheck
cristipaval Nov 1, 2023
ecdcca7
Merge branch 'main' of github.com:Expensify/App into beaman-removeRep…
Beamanator Nov 1, 2023
29bd51a
Re-remove managerEmail
Beamanator Nov 1, 2023
e6a9cba
Update version to 1.3.94-2
OSBotify Nov 1, 2023
0e97af0
Merge pull request #30401 from ZhenjaHorbach/amount-in-the-distance-e…
deetergp Nov 1, 2023
4ac2777
Merge pull request #30322 from namhihi237/fix-29900-tooltip-member-page
deetergp Nov 1, 2023
25ad2df
Update CONTRIBUTING.md
mallenexpensify Nov 1, 2023
30cffc4
Update CONTRIBUTING.md
mallenexpensify Nov 1, 2023
717e48d
Merge pull request #30320 from Expensify/greg-schroeder-patch-1
puneetlath Nov 1, 2023
3a46b61
Update CONTRIBUTING.md
mallenexpensify Nov 1, 2023
3db5ff4
Merge pull request #30736 from Expensify/mallenexpensify-patch-1
puneetlath Nov 1, 2023
3af0d9a
move table styles to article only
rushatgabhane Nov 2, 2023
51e76c6
Merge branch 'main' into remove-qrcode-folder
VickyStash Nov 2, 2023
8da24d8
Prevent user go to next step when press CMD+ENTER
hoangzinh Nov 2, 2023
ec1e215
Fix linter
hoangzinh Nov 2, 2023
3eb119d
Update version to 1.3.95-0
OSBotify Nov 2, 2023
2f4b98a
Early return if no waypoints were deleted
paultsimura Nov 2, 2023
d35caae
merge main
kacper-mikolajczak Nov 2, 2023
373d983
Merge branch 'Expensify:main' into fix--Android-pdf-scrolling-issue
HezekielT Nov 2, 2023
330d1ab
upgrade to `lottie-react-native` 6.4.0
kosmydel Nov 2, 2023
58513d5
fix package-lock.json
kosmydel Nov 2, 2023
6a9a9f2
fix package-lock.json
kosmydel Nov 2, 2023
a59a1ae
feat: add perf tests for composer
OlimpiaZurek Oct 31, 2023
3a94b35
CR fixes
OlimpiaZurek Nov 2, 2023
de7bf20
Merge pull request #30231 from tienifr/fix/regression-29888
mountiny Nov 2, 2023
87916f6
swap || to ??
kacper-mikolajczak Nov 2, 2023
8ad8345
Merge pull request #30443 from fabioh8010/ts/component/SVGImage
danieldoglas Nov 2, 2023
9870869
Merge pull request #30748 from rushatgabhane/fix-search
mountiny Nov 2, 2023
0543ad1
add waiter in UnreadIndicatorsTest
kacper-mikolajczak Nov 2, 2023
e36c722
Merge pull request #30515 from Expensify/beaman-removeReportManagerEmail
Gonals Nov 2, 2023
c79529a
Merge pull request #30050 from HezekielT/fix--Android-pdf-scrolling-i…
Gonals Nov 2, 2023
66c47f1
Merge pull request #30722 from Expensify/maddylewis-patch-11
mountiny Nov 2, 2023
a2844e0
Adds Violations Beta
cdanwards Nov 2, 2023
becbeb6
Merge pull request #30701 from eh2077/29827-fix-Android-inconsistent-…
MonilBhavsar Nov 2, 2023
7fffd19
Merge pull request #30279 from adamgrzybowski/@swm/limit-report-route…
mountiny Nov 2, 2023
8088a7b
Merge pull request #30633 from OlimpiaZurek/feat/27641/reassure-composer
mountiny Nov 2, 2023
3fd01be
Update src/languages/es.ts
grgia Nov 2, 2023
823717a
Merge branch 'main' into youssef_translation_fix
youssef-lr Nov 2, 2023
8b75fab
Merge pull request #30667 from Expensify/Christinadobrzyn-patch-1
grgia Nov 2, 2023
bad89e3
Merge pull request #28703 from Expensify/youssef_translation_fix
mountiny Nov 2, 2023
3ebe7bf
Update Approval-Workflows.md
zanyrenney Nov 2, 2023
b3e8fb2
Merge pull request #30715 from Expensify/maddylewis-patch-6
roryabraham Nov 2, 2023
b33ac36
Merge pull request #30785 from Expensify/zanyrenney-patch-2
roryabraham Nov 2, 2023
d966f65
fix 30211
c3024 Nov 2, 2023
a2bd366
Merge pull request #29988 from software-mansion-labs/ts-migration/Ful…
AndrewGable Nov 2, 2023
f7d7a9d
Merge pull request #30459 from rojiphil/30376-no-currency-for-tbd
luacmartins Nov 2, 2023
fdfa3f2
Merge pull request #28840 from ShogunFire/longPressForTaskPreviews
AndrewGable Nov 2, 2023
b049999
Merge pull request #30480 from VickyStash/remove-qrcode-folder
AndrewGable Nov 2, 2023
c959694
Merge branch 'main' into ts-migration/opacity-view-component
blazejkustra Nov 2, 2023
d1efd23
Shrink comments
blazejkustra Nov 2, 2023
3a8ea8f
added fundList in dependency
Pujan92 Nov 2, 2023
9cecdfa
Merge pull request #30657 from ishpaul777/fix/extra-gap-between-selec…
francoisl Nov 2, 2023
207266c
Merge pull request #28361 from Expensify/abekkala-patch-1
Nov 2, 2023
9ee3b4a
Merge pull request #29755 from software-mansion-labs/ts-migration/Saf…
Nov 2, 2023
d9da213
Merge pull request #30714 from Expensify/maddylewis-patch-5
jasperhuangg Nov 2, 2023
d7f97bb
Merge pull request #30719 from Expensify/maddylewis-patch-10
jasperhuangg Nov 2, 2023
25ed435
Merge pull request #30772 from software-mansion-labs/@kosmydel/upgrad…
roryabraham Nov 2, 2023
b6112c7
Merge pull request #30700 from Pujan92/fix/29767
jasperhuangg Nov 2, 2023
cd1c2bd
Merge pull request #30787 from c3024/fix-30211
chiragsalian Nov 2, 2023
11a7170
Merge pull request #30245 from Expensify/sonialiap-patch-1
aldo-expensify Nov 2, 2023
35c3b8a
Merge pull request #30302 from hoangzinh/df/27508
thienlnam Nov 2, 2023
f19217d
Merge pull request #30713 from Expensify/maddylewis-patch-4
aldo-expensify Nov 2, 2023
59317a4
Merge pull request #30354 from kacper-mikolajczak/perf/report-screen-…
thienlnam Nov 2, 2023
3d169ff
Merge pull request #30524 from Expensify/joekaufmanexpensify-patch-2
youssef-lr Nov 2, 2023
444a908
Merge pull request #30779 from infinitered/cdanwards/violations-betas
jasperhuangg Nov 2, 2023
fcb1fa5
Merge pull request #30670 from DylanDylann/fix/30539-app-crash-when-g…
stitesExpensify Nov 2, 2023
45c1297
Merge pull request #30754 from paultsimura/fix/30176-add-empty-stop
NikkiWines Nov 2, 2023
5ae80bd
Merge pull request #29248 from DylanDylann/fix/28925-public-room-anon…
stitesExpensify Nov 2, 2023
0822da0
fix issue where selected date is 1 before actual date
jasperhuangg Nov 2, 2023
2b88bba
use parseISO to ignore timezone checks
jasperhuangg Nov 2, 2023
60ec0cb
style
jasperhuangg Nov 2, 2023
bd1ecb3
Merge pull request #30810 from Expensify/jasper-fixDateRegression
NikkiWines Nov 2, 2023
6cc9053
Update version to 1.3.95-1
OSBotify Nov 3, 2023
d53a03e
Merge pull request #30711 from Expensify/maddylewis-patch-3
srikarparsi Nov 3, 2023
677729d
Bump onyx
ospfranco Nov 3, 2023
3e9b11c
check if participant is empty or not
bernhardoj Nov 3, 2023
3b77445
Merge branch 'main' of https://github.com/hurali97/Expensify-App into…
hurali97 Nov 3, 2023
18c219f
Merge pull request #28469 from hurali97/perf/refactor-heavy-operation…
mountiny Nov 3, 2023
1981886
fix regression copy to clipboard
saranshbalyan-1234 Nov 3, 2023
4322ad9
Merge pull request #30816 from bernhardoj/fix/30804-can't-create-sub-…
Beamanator Nov 3, 2023
85aad61
Update version to 1.3.95-2
OSBotify Nov 3, 2023
f498830
Merge pull request #30818 from saranshbalyan-1234/fix-regression-copy…
Beamanator Nov 3, 2023
c0657fa
Update version to 1.3.95-3
OSBotify Nov 3, 2023
2fec62f
fix: Green dot for money request is still displayed in spite of havin…
tienifr Nov 3, 2023
747b7d2
lint
tienifr Nov 3, 2023
9d99c41
Merge pull request #30830 from tienifr/fix/30824
Beamanator Nov 3, 2023
935096d
Update version to 1.3.95-4
OSBotify Nov 3, 2023
7ea89c9
Merge pull request #29857 from kubabutkiewicz/ts-migration/PopoverPro…
cristipaval Nov 3, 2023
cc6c08e
Merge pull request #29169 from margelo/osp/bump-onyx
Julesssss Nov 3, 2023
8d44cac
Merge pull request #30074 from software-mansion-labs/ts-migration/opa…
mountiny Nov 3, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
39 changes: 1 addition & 38 deletions .github/ISSUE_TEMPLATE/Standard.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,45 +42,8 @@ Which of our officially supported platforms is this issue occurring on?
- [ ] MacOS: Desktop

## Screenshots/Videos
<details>
<summary>Android: Native</summary>

<!-- add screenshots or videos here -->

</details>

<details>
<summary>Android: mWeb Chrome</summary>

<!-- add screenshots or videos here -->

</details>

<details>
<summary>iOS: Native</summary>

<!-- add screenshots or videos here -->

</details>

<details>
<summary>iOS: mWeb Safari</summary>

<!-- add screenshots or videos here -->

</details>

<details>
<summary>MacOS: Chrome / Safari</summary>

<!-- add screenshots or videos here -->

</details>

<details>
<summary>MacOS: Desktop</summary>

<!-- add screenshots or videos here -->
Add any screenshot/video evidence

</details>

Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/e2ePerformanceTests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -183,6 +183,7 @@ jobs:
file_artifacts: Customer Artifacts.zip
log_artifacts: debug.log
cleanup: true
timeout: 5400

- name: Print logs if run failed
if: failure()
Expand Down Expand Up @@ -213,6 +214,7 @@ jobs:
remote_src: false
file_artifacts: Customer Artifacts.zip
cleanup: true
timeout: 5400

- name: Unzip AWS Device Farm delta results
run: unzip "Customer Artifacts.zip" -d deltaResults
Expand Down
4 changes: 2 additions & 2 deletions android/app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,8 @@ android {
minSdkVersion rootProject.ext.minSdkVersion
targetSdkVersion rootProject.ext.targetSdkVersion
multiDexEnabled rootProject.ext.multiDexEnabled
versionCode 1001039400
versionName "1.3.94-0"
versionCode 1001039504
versionName "1.3.95-4"
}

flavorDimensions "default"
Expand Down
1 change: 0 additions & 1 deletion config/webpack/webpack.common.js
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,6 @@ const webpackConfig = ({envFile = '.env', platform = 'web'}) => ({
'react-native-config': 'react-web-config',
'react-native$': '@expensify/react-native-web',
'react-native-web': '@expensify/react-native-web',
'lottie-react-native': 'react-native-web-lottie',

// Module alias for web & desktop
// https://webpack.js.org/configuration/resolve/#resolvealias
Expand Down
41 changes: 5 additions & 36 deletions contributingGuides/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,12 +32,9 @@ This project and everyone participating in it is governed by the Expensify [Code
At this time, we are not hiring contractors in Crimea, North Korea, Russia, Iran, Cuba, or Syria.

## Slack channels
All contributors should be a member of **two** Slack channels:
All contributors should be a member of a shared Slack channel called [#expensify-open-source](https://expensify.slack.com/archives/C01GTK53T8Q) -- this channel is used to ask **general questions**, facilitate **discussions**, and make **feature requests**.

1. [#expensify-open-source](https://expensify.slack.com/archives/C01GTK53T8Q) -- used to ask **general questions**, facilitate **discussions**, and make **feature requests**.
2. [#expensify-bugs](https://expensify.slack.com/archives/C049HHMV9SM) -- used to discuss or report **bugs** specifically.

Before requesting an invite to Slack please ensure your Upwork account is active, since we only pay via Upwork (see [below](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#payment-for-contributions)). To request an invite to these two Slack channels, email contributors@expensify.com with the subject `Slack Channel Invites`. We'll send you an invite!
Before requesting an invite to Slack please ensure your Upwork account is active, since we only pay via Upwork (see [below](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#payment-for-contributions)). To request an invite to Slack, email contributors@expensify.com with the subject `Slack Channel Invites`. We'll send you an invite!

Note: Do not send direct messages to the Expensify team in Slack or Expensify Chat, they will not be able to respond.

Expand All @@ -47,30 +44,21 @@ Note: if you are hired for an Upwork job and have any job-specific questions, pl
If you've found a vulnerability, please email security@expensify.com with the subject `Vulnerability Report` instead of creating an issue.

## Payment for Contributions
We hire and pay external contributors via Upwork.com. If you'd like to be paid for contributing or reporting a bug, please create an Upwork account, apply for an available job in [GitHub](https://github.com/Expensify/App/issues?q=is%3Aopen+is%3Aissue+label%3A%22Help+Wanted%22), and finally apply for the job in Upwork once your proposal gets selected in GitHub. Please make sure your Upwork profile is **fully verified** before applying, otherwise you run the risk of not being paid. If you think your compensation should be increased for a specific job, you can request a reevaluation by commenting in the Github issue where the Upwork job was posted.
We hire and pay external contributors via Upwork.com. If you'd like to be paid for contributing, please create an Upwork account, apply for an available job in [GitHub](https://github.com/Expensify/App/issues?q=is%3Aopen+is%3Aissue+label%3A%22Help+Wanted%22), and finally apply for the job in Upwork once your proposal gets selected in GitHub. Please make sure your Upwork profile is **fully verified** before applying, otherwise you run the risk of not being paid. If you think your compensation should be increased for a specific job, you can request a reevaluation by commenting in the Github issue where the Upwork job was posted.

Payment for your contributions and bug reports will be made no less than 7 days after the pull request is deployed to production to allow for [regression](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#regressions) testing. If you have not received payment after 8 days of the PR being deployed to production, and there are no [regressions](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#regressions), please add a comment to the issue mentioning the BugZero team member (Look for the melvin-bot "Triggered auto assignment to... (`Bug`)" to see who this is).
Payment for your contributions will be made no less than 7 days after the pull request is deployed to production to allow for [regression](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#regressions) testing. If you have not received payment after 8 days of the PR being deployed to production, and there are no [regressions](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#regressions), please add a comment to the issue mentioning the BugZero team member (Look for the melvin-bot "Triggered auto assignment to... (`Bug`)" to see who this is).

New contributors are limited to working on one job at a time, however experienced contributors may work on numerous jobs simultaneously.

Please be aware that compensation for any support in solving an issue is provided **entirely at Expensify’s discretion**. Personal time or resources applied towards investigating a proposal **will not guarantee compensation**. Compensation is only guaranteed to those who **[propose a solution and get hired for that job](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#propose-a-solution-for-the-job)**. We understand there may be cases where a selected proposal may take inspiration from a previous proposal. Unfortunately, it’s not possible for us to evaluate every individual case and we have no process that can efficiently do so. Issues with higher rewards come with higher risk factors so try to keep things civil and make the best proposal you can. Once again, **any information provided may not necessarily lead to you getting hired for that issue or compensated in any way.**

**Important:** Payment amounts are variable, dependent on when your PR is merged and if there are any [regressions](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#regressions). Your PR will be reviewed by a [Contributor+ (C+)](https://github.com/Expensify/App/blob/main/contributingGuides/HOW_TO_BECOME_A_CONTRIBUTOR_PLUS.md). team member and an internal engineer. All tests must pass and all code must pass lint checks before a merge.

**Payment timelines** are based on the day and timestamp the contributor is assigned to the Github issue by an Expensify employee:
- Merged PR within 3 business days (72 hours) - 50% **bonus**
- Merged PR within 6 business days (144 hours) - 0% bonus
- Merged PR within 9 business days (216 hours) - 50% **penalty**
- No PR within 12 business days - **Contract terminated**

We specify exact hours to make sure we can clearly decide what is eligible for the bonus given our team is global and contributors span across all the timezones.
**Important:** Payment amounts are variable, dependent on if there are any [regressions](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#regressions). Your PR will be reviewed by a [Contributor+ (C+)](https://github.com/Expensify/App/blob/main/contributingGuides/HOW_TO_BECOME_A_CONTRIBUTOR_PLUS.md) team member and an internal engineer. All tests must pass and all code must pass lint checks before a merge.

### Regressions

If a PR causes a regression at any point within the regression period (starting when the code is merged and ending 168 hours (that's 7 days) after being deployed to production):
- payments will be issued 7 days after all regressions are fixed (ie: deployed to production)
- a 50% penalty will be applied to the Contributor and [Contributor+](https://github.com/Expensify/App/blob/main/contributingGuides/HOW_TO_BECOME_A_CONTRIBUTOR_PLUS.md) for each regression on an issue
- the assigned Contributor and [Contributor+](https://github.com/Expensify/App/blob/main/contributingGuides/HOW_TO_BECOME_A_CONTRIBUTOR_PLUS.md) are not eligible for the 50% urgency bonus

The 168 hours (aka 7 days) will be measured by calculating the time between when the PR is merged, and when a bug is posted to the #expensify-bugs Slack channel.

Expand All @@ -80,25 +68,6 @@ A job could be fixing a bug or working on a new feature. There are two ways you
#### Finding a job that Expensify posted
This is the most common scenario for contributors. The Expensify team posts new jobs to the Upwork job list [here](https://www.upwork.com/ab/jobs/search/?q=Expensify%20React%20Native&sort=recency&user_location_match=2) (you must be signed in to Upwork to view jobs). Each job in Upwork has a corresponding GitHub issue, which will include instructions to follow. You can also view all open jobs in the Expensify/App GH repository by searching for GH issues with the [`Help Wanted` label](https://github.com/Expensify/App/issues?q=is%3Aopen+is%3Aissue+label%3A%22Help+Wanted%22). Lastly, you can follow the [@ExpensifyOSS](https://twitter.com/ExpensifyOSS) Twitter account to see a live feed of jobs that are posted.

#### Raising jobs and bugs
It’s possible that you found a new bug that we haven’t posted as a job to the [GitHub repository](https://github.com/Expensify/App/issues?q=is%3Aissue). This is an opportunity to raise it and claim the bug bounty. If it's a valid bug that we choose to resolve by deploying it to production — either internally or via an external contributor — then we will compensate you $50 for identifying the bug (we do not compensate for reporting new feature requests). If the bug is fixed by a PR that is not associated with your bug report, then you will not be eligible for the corresponding compensation unless you can find the PR that fixed it and prove your bug report came first.
- Note: If you get assigned the job you proposed **and** you complete the job, this $50 for identifying the improvement is *in addition to* the reward you will be paid for completing the job.
- Note about proposed bugs: Expensify has the right not to pay the $50 reward if the suggested bug has already been reported. Following, if more than one contributor proposes the same bug, the contributor who posted it first in the [#expensify-bugs](https://expensify.slack.com/archives/C049HHMV9SM) Slack channel is the one who is eligible for the bonus.
- Note: whilst you may optionally propose a solution for that job on Slack, solutions are ultimately reviewed in GitHub. The onus is on you to propose the solution on GitHub, and/or ensure the issue creator will include a link to your proposal.

Please follow these steps to propose a job or raise a bug:

1. Check to ensure a GH issue does not already exist for this job in the [New Expensify Issue list](https://github.com/Expensify/App/issues).
2. Check to ensure the `Bug:` or `Feature Request:` was not already posted in Slack (specifically the #expensify-bugs or #expensify-open-source [Slack channels](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#slack-channels)). Use your best judgement by searching for similar titles, words and issue descriptions.
3. If your bug or new feature matches with an existing issue, please comment on that Slack thread or GitHub issue with your findings if you think it will help solve the issue.
4. If there is no existing GitHub issue or Upwork job, check if the issue is happening on prod (as opposed to only happening on dev)
5. If the issue is just in dev then it means it's a new issue and has not been deployed to production. In this case, you should try to find the offending PR and comment in the issue tied to the PR and ask the assigned users to add the `DeployBlockerCash` label. If you can't find it, follow the reporting instructions in the next item, but note that the issue is a regression only found in dev and not in prod.
6. If the issue happens in main, staging, or production then report the issue(s) in the #expensify-bugs Slack channel, using the report bug workflow. You can do this by clicking 'Workflow > report Bug', or typing `/Report bug`. View [this guide](https://github.com/Expensify/App/blob/main/contributingGuides/HOW_TO_CREATE_A_PLAN.md) for help creating a plan when proposing a feature request. Please verify the bug's presence on **every** platform mentioned in the bug report template, and confirm this with a screen recording..
- **Important note/reminder**: never share any information pertaining to a customer of Expensify when describing the bug. This includes, and is not limited to, a customer's name, email, and contact information.
7. The Applause team will review your job proposal in the appropriate slack channel. If you've provided a quality proposal that we choose to implement, a GitHub issue will be created and your Slack handle will be included in the original post after `Issue reported by:`
8. If an external contributor other than yourself is hired to work on the issue, you will also be hired for the same job in Upwork to receive your payout. No additional work is required. If the issue is fixed internally, a dedicated job will be created to hire and pay you after the issue is fixed.
9. Payment will be made 7 days after code is deployed to production if there are no regressions. If a regression is discovered, payment will be issued 7 days after all regressions are fixed.

>**Note:** Our problem solving approach at Expensify is to focus on high value problems and avoid small optimizations with results that are difficult to measure. We also prefer to identify and solve problems at their root. Given that, please ensure all proposed jobs fix a specific problem in a measurable way with evidence so they are easy to evaluate. Here's an example of a good problem/solution:
>
>**Problem:** The app start up time has regressed because we introduced "New Feature" in PR #12345 and is now 1042ms slower because `SomeComponent` is re-rendering 42 times.
Expand Down
3 changes: 0 additions & 3 deletions docs/_includes/footer.html
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,6 @@ <h3>Features</h3>
<li>
<a href="https://use.expensify.com/invoices" alt="Invoicing" target="_blank">Invoicing</a>
</li>
<li>
<a href="https://use.expensify.com/cpa-card" alt="CPA Card" target="_blank">CPA Card</a>
</li>
<li>
<a href="https://use.expensify.com/payroll" alt="Payroll" target="_blank">Payroll</a>
</li>
Expand Down
67 changes: 37 additions & 30 deletions docs/_sass/_main.scss
Original file line number Diff line number Diff line change
Expand Up @@ -35,31 +35,8 @@ html {
}

table {
margin-bottom: 20px;
border-spacing: 0;
border-collapse: collapse;
border-radius: 8px;

// Box shadow is used here because border-radius and border-collapse don't work together. It leads to double borders.
// https://stackoverflow.com/questions/628301/the-border-radius-property-and-border-collapsecollapse-dont-mix-how-can-i-use
border-style: hidden;
box-shadow: 0 0 0 1px $color-green-borders;
}

th:first-child {
border-top-left-radius: 8px;
}

th:last-child {
border-top-right-radius: 8px;
}

tr:last-child > td:first-child {
border-bottom-left-radius: 8px;
}

tr:last-child > td:last-child {
border-bottom-right-radius: 8px;
}

caption,
Expand All @@ -68,13 +45,6 @@ td {
text-align: left;
font-weight: 400;
vertical-align: middle;
padding: 6px 13px;
border: 1px solid $color-green-borders;
}

thead tr th {
font-weight: bold;
background-color: $color-green-highlightBG;
}

q,
Expand Down Expand Up @@ -395,6 +365,43 @@ button {
}
}

table {
margin-bottom: 20px;
border-radius: 8px;

// Box shadow is used here because border-radius and border-collapse don't work together. It leads to double borders.
// https://stackoverflow.com/questions/628301/the-border-radius-property-and-border-collapsecollapse-dont-mix-how-can-i-use
border-style: hidden;
box-shadow: 0 0 0 1px $color-green-borders;
}

th:first-child {
border-top-left-radius: 8px;
}

th:last-child {
border-top-right-radius: 8px;
}

tr:last-child > td:first-child {
border-bottom-left-radius: 8px;
}

tr:last-child > td:last-child {
border-bottom-right-radius: 8px;
}

th,
td {
padding: 6px 13px;
border: 1px solid $color-green-borders;
}

thead tr th {
font-weight: bold;
background-color: $color-green-highlightBG;
}

.img-wrap {
display: flex;
justify-content: space-around;
Expand Down
Loading
Loading