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

E2e/add typing tests #17

Closed
wants to merge 255 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
255 commits
Select commit Hold shift + click to select a range
70e7830
Merge branch 'main' into fix/24366-iou-transaction
bernhardoj Sep 15, 2023
3f41979
Merge branch 'main' of https://github.com/tienifr/App into fix/26990-…
tienifr Sep 18, 2023
c6fd83e
update comment
tienifr Sep 18, 2023
e608638
Migrate policy utils lib
BartoszGrajdek Sep 20, 2023
4785593
Remove the categories beta
yuwenmemon Sep 27, 2023
2672054
Merge branch 'main' of github.com:Expensify/App into yuwen-removeCatBeta
yuwenmemon Sep 27, 2023
7e4e58f
Fix JS error
yuwenmemon Sep 27, 2023
ca550a5
Merge branch 'main' of github.com:Expensify/App into yuwen-removeCatBeta
yuwenmemon Sep 27, 2023
f38e0e5
Initial work
blazejkustra Sep 27, 2023
12f3816
Merge branch 'main' of github.com:Expensify/App into yuwen-removeCatBeta
yuwenmemon Sep 28, 2023
b7fceda
Fix conflicts
yuwenmemon Sep 28, 2023
8113ebf
Type fixes
BartoszGrajdek Sep 29, 2023
466814e
Resolve merge conflicts
BartoszGrajdek Sep 29, 2023
837bd88
Migrate PolicyUtils to TS
BartoszGrajdek Oct 2, 2023
e464283
Merge remote-tracking branch 'origin/main' into ts-migration/policy-u…
BartoszGrajdek Oct 2, 2023
f42364d
[TS migration] Migrate 'SidebarUtils.js' lib to TypeScript
blazejkustra Oct 2, 2023
08da8c0
Self review the code
blazejkustra Oct 3, 2023
bf4c2f1
Merge branch 'main' into ts-migration/SidebarUtils
blazejkustra Oct 3, 2023
bbcdbd1
Changes after review of PolicyUtils
BartoszGrajdek Oct 3, 2023
334061b
Update Business-Bank-Accounts-USD.md
OfstadC Oct 6, 2023
3c41ef5
Fixes in onyxkeys
BartoszGrajdek Oct 11, 2023
bb6cd81
migrate useReportScrollManager and ReportScreenContext to TypeScript
JKobrynski Oct 11, 2023
df77b50
Merge branch 'main' into fix/24366-iou-transaction
bernhardoj Oct 12, 2023
5248bc6
Form validation fixes
kowczarz Oct 5, 2023
5b4a419
Form docs improvements
kowczarz Oct 6, 2023
bd87379
Fix prop types
kowczarz Oct 6, 2023
10ea700
Remove console log
kowczarz Oct 6, 2023
9094b98
Merge branch 'main' into migrateUseReportScrollManagerToTypeScript
JKobrynski Oct 12, 2023
56b46a1
extract return values to a separate type
JKobrynski Oct 12, 2023
568767c
RTL Text gets renderd properly
HardikChoudhary24 Oct 12, 2023
f83a33f
Remove dead code
BartoszGrajdek Oct 12, 2023
eab18e6
Refactor roomNameInput
kowczarz Oct 12, 2023
40a6da2
Refactor RoomNamePage
kowczarz Oct 12, 2023
e35bddc
Refactor WorkspaceNewRoomPage
kowczarz Oct 12, 2023
56abdce
Merge branch 'main' into ts-migration/SidebarUtils
blazejkustra Oct 12, 2023
1715578
Address code review
blazejkustra Oct 12, 2023
2a25290
Add errors to ReportActionBase
blazejkustra Oct 12, 2023
0e7d40d
Add optional chaining to fix tests
blazejkustra Oct 12, 2023
ccd959e
Fix remaining tests
blazejkustra Oct 12, 2023
4f6af3d
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
9c2335d
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
31c109c
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
81842f8
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
a04eb50
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
266928f
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
ba56c91
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
e8c91ef
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
e018035
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
bf4504d
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
e929211
Update docs/articles/expensify-classic/bank-accounts-and-credit-cards…
OfstadC Oct 12, 2023
d13cfb8
RTL text gets rendered properly for safari
HardikChoudhary24 Oct 13, 2023
5c4ea8e
Merge branch 'main' of https://github.com/Expensify/App into fix/2814…
HardikChoudhary24 Oct 13, 2023
669410f
RTL text gets rendered properly for safari
HardikChoudhary24 Oct 13, 2023
010b47b
Quick fixes for policy utils
BartoszGrajdek Oct 16, 2023
68a0ee7
add platform specific types
JKobrynski Oct 16, 2023
2e12b63
Merge remote-tracking branch 'expensify/main' into from-migration/roo…
kowczarz Oct 16, 2023
1fd92cc
Post-merge fixes
kowczarz Oct 16, 2023
ba1a78d
Prettier fixes
kowczarz Oct 16, 2023
1ccbae3
improve scrolling to index, minor fixes
JKobrynski Oct 16, 2023
24f858d
Merge branch 'main' into ts-migration/SidebarUtils
blazejkustra Oct 16, 2023
a8886c9
Suggested changes executed
HardikChoudhary24 Oct 17, 2023
7f6c9f0
[TS migration] Migrate 'FormScrollView.js' component to TypeScript
blazejkustra Oct 17, 2023
fefdb2b
Add value parser to the new form
kowczarz Oct 17, 2023
e05b537
Remove native room name input
kowczarz Oct 17, 2023
eff3e88
Update room name input
kowczarz Oct 17, 2023
187fa7d
Merge remote-tracking branch 'expensify/main' into from-migration/roo…
kowczarz Oct 17, 2023
c22c8c9
Merge branch 'main' into ts-migration/SidebarUtils
blazejkustra Oct 17, 2023
f467ac2
remove platform specific type file, add default export
JKobrynski Oct 17, 2023
398b842
Prettier
kowczarz Oct 17, 2023
b5e45fc
chore: add flashlist dependency
adhorodyski Oct 7, 2023
22e6de2
chore: add flashlist to the jest setup
adhorodyski Oct 7, 2023
0de9d9d
refactor: migrate PaymentMethodList to use FlashList
adhorodyski Oct 9, 2023
6f09876
fix: lint error with missing props declaration
adhorodyski Oct 9, 2023
188326e
fix: improved estimated_item_size for list config, added jsdoc comments
adhorodyski Oct 9, 2023
646fb82
Merge branch 'main' into fix/26990
tienifr Oct 18, 2023
2c5815f
fix warning in PreRenderer
tienifr Oct 18, 2023
87f112b
remove preventDefault
tienifr Oct 18, 2023
44d9c4d
Merge branch 'Expensify:main' into refactor/28902/flashlist-migration
adhorodyski Oct 18, 2023
c259f7d
Merge branch 'main' of https://github.com/tienifr/App into fix/26990-…
tienifr Oct 18, 2023
505869d
remove feature detection
tienifr Oct 18, 2023
acba999
Resolve merge conflicts
BartoszGrajdek Oct 18, 2023
8d20b1e
Resolve merge conflicts in onyx/index.ts
BartoszGrajdek Oct 18, 2023
edd40b9
Merge remote-tracking branch 'expensify/main' into from-migration/roo…
kowczarz Oct 18, 2023
bd686b2
Code review changes
kowczarz Oct 18, 2023
947220b
Adjust after internal review
blazejkustra Oct 18, 2023
abee722
Resoved merge conflicts
HardikChoudhary24 Oct 18, 2023
9ba0995
Merge branch 'main' of https://github.com/tienifr/App into fix/26990-…
tienifr Oct 18, 2023
425de8a
ts migration
tienifr Oct 18, 2023
a1c0e24
Merge branch 'Expensify:main' into refactor/28902/flashlist-migration
adhorodyski Oct 19, 2023
5ad7f00
Remove unused KeyboardDismissingFlatList component
VickyStash Oct 19, 2023
da94edd
Merge branch 'Expensify:main' into refactor/28902/flashlist-migration
adhorodyski Oct 19, 2023
bbb35e8
Merge branch 'main' into ts-migration/SidebarUtils
blazejkustra Oct 19, 2023
b4c0c1d
Remove hidden reports like threads from the global unread count
marcaaron Oct 19, 2023
e63abb7
Merge remote-tracking branch 'upstream/main' into fix/28149-wrong-RTL…
HardikChoudhary24 Oct 19, 2023
acd395e
Suggested changes applied
HardikChoudhary24 Oct 19, 2023
8084b25
Switch to &&
marcaaron Oct 19, 2023
79fdee9
Add checkout to setup for OSBotify
justinpersaud Oct 19, 2023
411b053
add step to check if gpg key is present
justinpersaud Oct 19, 2023
f11a463
add conditional to checkout
justinpersaud Oct 19, 2023
50456d6
simplify check
justinpersaud Oct 19, 2023
233d703
Bring back test
blazejkustra Oct 20, 2023
f137763
Add comments to onyx types
blazejkustra Oct 20, 2023
6aeeacf
fix clip icon briefly showing on attachment preview
mkhutornyi Oct 20, 2023
af532f4
Merge remote-tracking branch 'upstream/main' into fix/28149-wrong-RTL…
HardikChoudhary24 Oct 21, 2023
1dd55f1
Separate module convertToLTRForComposer created
HardikChoudhary24 Oct 21, 2023
389f982
Platform import removed
HardikChoudhary24 Oct 21, 2023
f921458
Remove PolicyUtils.ts
BartoszGrajdek Oct 23, 2023
db43443
Rename PolicyUtils
BartoszGrajdek Oct 23, 2023
560b8ca
Migrate PolicyUtils to TS
BartoszGrajdek Oct 23, 2023
c133db8
Use FormProvider in NewContactMethodPage
cdOut Oct 18, 2023
425ecb5
Use FormProvider in StatusSetPage
cdOut Oct 19, 2023
a660653
Merge branch 'Expensify:main' into refactor/28902/flashlist-migration
adhorodyski Oct 23, 2023
b9138d7
fix: flashlist container styles
adhorodyski Oct 19, 2023
7a9e846
fix: move flashlist container styles to a wrapper
adhorodyski Oct 19, 2023
7ce37a6
chore: add minHeight to the flashlist wrapper View to prevent measure…
adhorodyski Oct 23, 2023
5b12a46
Improved initial state in useRefs - changed useRefs to more suitable …
sebryu Oct 15, 2023
b0be5dd
Refactored storing useMemo first computation to new hook - useInitial…
sebryu Oct 20, 2023
10952f2
Eslint fix
sebryu Oct 20, 2023
c1a1acf
Eslint fix
sebryu Oct 23, 2023
8e7a65b
Merge remote-tracking branch 'upstream/main' into fix/28149-wrong-RTL…
HardikChoudhary24 Oct 23, 2023
ec7860f
rule name specified
HardikChoudhary24 Oct 23, 2023
e4364b3
Update semver on desktop
pecanoro Oct 23, 2023
8de04a6
Update to 6.0 (good luck)
pecanoro Oct 23, 2023
48dbc7b
reduce label size and remove placeholder in new chat room
ahmedGaber93 Oct 23, 2023
8d81cb6
move the magic code input to a fixed footer outside the scrollview
akinwale Oct 24, 2023
0bbf2d1
[TS migration] Migrate 'InlineSystemMessage.js' component
VickyStash Oct 24, 2023
8feada1
Removed dependency array from useInitialValue call
sebryu Oct 24, 2023
0bcf5ea
wip: adding adb commands
hannojg Oct 24, 2023
dcc6a27
Merge branch 'main' into fix/26990
tienifr Oct 24, 2023
df8c11c
Resolve merge conflicts
BartoszGrajdek Oct 24, 2023
475944c
Remove unused ExpensifyCashLogo component
VickyStash Oct 24, 2023
947b43e
remove branch args
hannojg Oct 24, 2023
51d2ffc
add package scripts to help run in dev mode
hannojg Oct 24, 2023
4347739
fix docs
hannojg Oct 24, 2023
d68434e
Merge remote-tracking branch 'expensify/main' into from-migration/roo…
kowczarz Oct 24, 2023
47647d5
Merge branch 'main' into ts-migration/SidebarUtils
blazejkustra Oct 24, 2023
a224678
Remove isLastMessageDeletedParentAction from getOptionData
blazejkustra Oct 24, 2023
10b6eb1
Merge branch 'main' into form-migration/StatusSetPage
cdOut Oct 24, 2023
e052333
Merge branch 'main' into form-migration/NewContactMethodPage
cdOut Oct 24, 2023
951c92a
update documentation
hannojg Oct 24, 2023
e16b218
doc: improve wording
hannojg Oct 24, 2023
3762292
remove note
hannojg Oct 24, 2023
4a50dbc
fix: 29950 Reload on title and description of task field does not foc…
tienifr Oct 24, 2023
3212c6f
fix: replace emojis on mount(both the main composer and edit composer)
sangar-1028 Oct 24, 2023
6692d60
fix: call update draft when locale changes
sangar-1028 Oct 24, 2023
70329db
fix: lint error
sangar-1028 Oct 24, 2023
12b88e3
fix: prettier
sangar-1028 Oct 24, 2023
05ea988
fix: update parent action only if the action has no visible children
sangar-1028 Oct 24, 2023
cc43bd4
Skip reassure tests for unrelated changes
roryabraham Oct 24, 2023
7522648
Also skip any md files
roryabraham Oct 24, 2023
d2f7274
Use built-in paths-ignore option
roryabraham Oct 24, 2023
dd2b24f
Skip lint for PRs that only affect unlinted files
roryabraham Oct 25, 2023
f6095d7
Skip tests for unaffected PRs
roryabraham Oct 25, 2023
ce985ef
Skip typecheck for unaffected PRs
roryabraham Oct 25, 2023
73ac84a
Skip deploy of app for unrelated changes
roryabraham Oct 25, 2023
575f32c
Update @xmldom/xmldom to avoid critical vulnerabilities
pecanoro Oct 25, 2023
f135ee3
Move welcome comment to separate workflow
roryabraham Oct 25, 2023
cd3bc74
Remove unnecessary token
roryabraham Oct 25, 2023
8351917
wip adding test
hannojg Oct 25, 2023
abeb8f1
add missing config entry
hannojg Oct 25, 2023
dad3957
fix api mock not returning data when none onyx request
hannojg Oct 25, 2023
47f7e73
wip: navigate to report
hannojg Oct 25, 2023
b7c5aec
Merge branch 'main' into ts-migration/SidebarUtils
blazejkustra Oct 25, 2023
8d42d7f
Resolve conflicts
blazejkustra Oct 25, 2023
9b76129
Merge branch 'main' into ts-migration/FormScrollView
blazejkustra Oct 25, 2023
e35656b
add more mock data
hannojg Oct 25, 2023
8816307
add mock for read newest action
hannojg Oct 25, 2023
9655acd
add mock for ReconnectToReport
hannojg Oct 25, 2023
7e06bd7
Merge branch 'main' into Sebryu-performance-initial-ref-optimization
sebryu Oct 25, 2023
a37be42
Merge remote-tracking branch 'expensify/main' into from-migration/roo…
kowczarz Oct 25, 2023
6edeecf
Remove empty file
kowczarz Oct 25, 2023
713bdf0
Merge branch 'main' into Rory-SkipTestsForUnrelatedChanges
roryabraham Oct 25, 2023
76be1d5
Remove outdated needs
roryabraham Oct 25, 2023
e807859
Just use github.token for welcome comment
roryabraham Oct 25, 2023
fb0899f
Memo HeaderView
janicduplessis Oct 26, 2023
46170f8
Merge branch 'main' into Sebryu-performance-initial-ref-optimization
sebryu Oct 26, 2023
6007e60
Update ReportScreen.js
janicduplessis Oct 26, 2023
cd851f5
Merge pull request #29643 from software-mansion-labs/Sebryu-performan…
mountiny Oct 26, 2023
e44467e
fix: update style if icon is in button
sangar-1028 Oct 26, 2023
2c52130
Merge pull request #27429 from tienifr/fix/26990-warning-non-passive-…
danieldoglas Oct 26, 2023
414d591
Merge pull request #30213 from ahmedGaber93/issue-29923
danieldoglas Oct 26, 2023
998f8c6
fix: prettier
sangar-1028 Oct 26, 2023
685f8f7
resolve conflicts
BartoszGrajdek Oct 26, 2023
f0b7aea
Fix wrong ref
kowczarz Oct 26, 2023
02c6f00
Merge branch 'Expensify:main' into refactor/28902/flashlist-migration
adhorodyski Oct 26, 2023
c285013
chore: remove a redundant rename on the isOffline param
adhorodyski Oct 26, 2023
3d6ad2f
chore: reuse global paymentMethodHeight value on the list
adhorodyski Oct 26, 2023
d656c9c
update the screenshots template in the reviewer checklist
akinwale Oct 26, 2023
565ea7c
Merge remote-tracking branch 'expensify/main' into from-migration/roo…
kowczarz Oct 26, 2023
d20f983
Merge main into branch
puneetlath Oct 26, 2023
0d3830e
Remove one more case of canUseCategories
puneetlath Oct 26, 2023
a44824d
Merge pull request #29443 from software-mansion-labs/ts-migration/wit…
puneetlath Oct 26, 2023
f3d70cc
Prettier fix
puneetlath Oct 26, 2023
19077c1
Fix wrong ref passing
kowczarz Oct 26, 2023
ea9b46e
Merge pull request #30440 from akinwale/task-reviewer-checklist
Julesssss Oct 26, 2023
2d52929
Merge pull request #30266 from tienifr/fix/29905
rlinoz Oct 26, 2023
91642d2
Merge pull request #30234 from VickyStash/ts-migration/inlineSystemMe…
Gonals Oct 26, 2023
a67253b
Merge pull request #30406 from janicduplessis/@janic/header-memo
Gonals Oct 26, 2023
7e72ab7
Update Card-Settings.md
gabrielessner Oct 26, 2023
8c76d2a
Merge pull request #28296 from Expensify/yuwen-removeCatBeta
pecanoro Oct 26, 2023
9e50be4
Merge pull request #29869 from software-mansion-labs/form-migration/N…
luacmartins Oct 26, 2023
dae3c5a
Merge pull request #29952 from software-mansion-labs/form-migration/S…
luacmartins Oct 26, 2023
92677e3
Merge pull request #29095 from adhorodyski/refactor/28902/flashlist-m…
roryabraham Oct 26, 2023
ccf0b9f
Fix propType
kowczarz Oct 26, 2023
020ba40
Merge pull request #30417 from s-alves10/fix/issue-30400
chiragsalian Oct 26, 2023
99f4fe6
Merge pull request #29771 from software-mansion-labs/from-migration/r…
luacmartins Oct 26, 2023
9802795
Merge pull request #28596 from software-mansion-labs/ts-migration/Sid…
bondydaa Oct 26, 2023
6c2e087
Merge pull request #27306 from bernhardoj/fix/24366-iou-transaction
bondydaa Oct 26, 2023
d46a691
Update version to 1.3.91-7
OSBotify Oct 26, 2023
e47a256
Merge pull request #30013 from Expensify/marcaaron-fixGlobalUnreadInd…
marcaaron Oct 26, 2023
294afaf
Merge pull request #30321 from Expensify/rocio-xmldom
marcaaron Oct 26, 2023
a5e1e3f
Merge pull request #30317 from s-alves10/fix/issue-26012
thienlnam Oct 26, 2023
6b9fc22
Add autoScrollToTopThreshold
roryabraham Oct 26, 2023
b335f2f
Merge pull request #30301 from s-alves10/fix/issue-28265
dangrous Oct 26, 2023
f06f1d6
Merge pull request #29989 from VickyStash/ts-migration/keyboardDismis…
thienlnam Oct 26, 2023
ff89240
Merge pull request #30450 from Expensify/gabrielessner-patch-1
stitesExpensify Oct 26, 2023
e824526
[ImgBot] Optimize images
ImgBotApp Oct 26, 2023
a75baa1
Merge pull request #30463 from Expensify/imgbot
github-actions[bot] Oct 26, 2023
b644611
Merge pull request #29757 from software-mansion-labs/ts-migration/For…
nkuoch Oct 26, 2023
4809055
Merge pull request #30241 from VickyStash/ts-migration/expensifyCashL…
NikkiWines Oct 26, 2023
a4ab410
Merge pull request #30227 from akinwale/task-26695
roryabraham Oct 26, 2023
6863ca6
Merge pull request #30461 from Expensify/Rory-FixAndroidScrollRegression
marcaaron Oct 26, 2023
53699cf
Update version to 1.3.91-8
OSBotify Oct 26, 2023
546f27e
Merge pull request #30029 from Expensify/jpersaud_osbotify_updates
roryabraham Oct 26, 2023
ac5470e
Merge branch 'main' into Rory-SkipTestsForUnrelatedChanges
roryabraham Oct 26, 2023
835f7f7
Add test change in JS file - tests and lint should run
roryabraham Oct 26, 2023
f9fa1db
Ok now only a ts change
roryabraham Oct 26, 2023
7cacd8e
Consistently use double-wildcarding
roryabraham Oct 26, 2023
bf72a7c
remove test ts change
roryabraham Oct 26, 2023
ac41cd8
Test markdown change
roryabraham Oct 26, 2023
ca0235a
Test markdown change in docs folder
roryabraham Oct 26, 2023
2e3037d
Remove test markdown change
roryabraham Oct 26, 2023
e51b21b
Merge pull request #30099 from mkhutornyi/fix-28036
jasperhuangg Oct 26, 2023
33f15e8
Separate out shell script linting
roryabraham Oct 26, 2023
0c7c06f
Ignore shell scripts in reassurePerformanceTests
roryabraham Oct 26, 2023
6aff07f
Update version to 1.3.92-0
OSBotify Oct 26, 2023
ebff489
Merge pull request #30315 from Expensify/Rory-SkipTestsForUnrelatedCh…
AndrewGable Oct 26, 2023
c9abaf6
Merge pull request #28587 from software-mansion-labs/ts-migration/pol…
AndrewGable Oct 26, 2023
e51f746
Merge pull request #29014 from Expensify/OfstadC-patch-1
chiragsalian Oct 26, 2023
83e2b97
Merge pull request #30186 from Expensify/rocio-DesktopSemver
tylerkaraszewski Oct 26, 2023
b15fc21
Merge pull request #29315 from JKobrynski/migrateUseReportScrollManag…
roryabraham Oct 26, 2023
eef8347
Merge pull request #29434 from HardikChoudhary24/fix/28149-wrong-RTL-…
neil-marcellini Oct 27, 2023
8dfd4ab
wip: having a .e2e.js component
hannojg Oct 27, 2023
337bc06
implemented typing into composer
hannojg Oct 27, 2023
953b91d
send rerender results to client
hannojg Oct 27, 2023
602bdbd
remove the connotation of "duration"
hannojg Oct 27, 2023
8152008
add clearing of input
hannojg Oct 27, 2023
9ea214d
fix warnings thrown
hannojg Oct 27, 2023
31b677e
remove unused native ID
hannojg Oct 27, 2023
8ea9e07
Merge branch 'main' of github.com:Expensify/App into e2e/add-typing-t…
hannojg Oct 27, 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
15 changes: 15 additions & 0 deletions .github/actions/composite/setupGitForOSBotifyApp/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,21 @@ outputs:
runs:
using: composite
steps:
- name: Check if gpg encrypted private key is present
id: key_check
shell: bash
run: |
if [[ -f .github/workflows/OSBotify-private-key.asc.gpg ]]; then
echo "::set-output name=key_exists::true"
fi

- name: Checkout
uses: actions/checkout@v3
if: steps.key_check.outputs.key_exists != 'true'
with:
sparse-checkout: |
.github

- name: Decrypt OSBotify GPG key
run: cd .github/workflows && gpg --quiet --batch --yes --decrypt --passphrase=${{ inputs.GPG_PASSPHRASE }} --output OSBotify-private-key.asc OSBotify-private-key.asc.gpg
shell: bash
Expand Down
6 changes: 2 additions & 4 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,15 @@ on:
pull_request:
types: [opened, synchronize]
branches-ignore: [staging, production]
paths: ['**.js', '**.ts', '**.tsx']

jobs:
lint:
if: ${{ github.actor != 'OSBotify' || github.event_name == 'workflow_call' }}
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8
uses: actions/checkout@v3

- name: Setup Node
uses: Expensify/App/.github/actions/composite/setupNode@main
Expand All @@ -22,9 +23,6 @@ jobs:
env:
CI: true

- name: Lint shell scripts with ShellCheck
run: npm run shellcheck

- name: Verify there's no Prettier diff
run: |
npm run prettier -- --loglevel silent
Expand Down
69 changes: 2 additions & 67 deletions .github/workflows/preDeploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ name: Process new code merged to main

on:
push:
branches:
- main
branches: [main]
paths-ignore: [docs/**, contributingGuides/**, jest/**, tests/**, workflow_tests/**]

jobs:
typecheck:
Expand Down Expand Up @@ -112,71 +112,6 @@ jobs:
with:
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}

# Check if actor is member of Expensify organization by looking for Expensify/expensify team
isExpensifyEmployee:
runs-on: ubuntu-latest

outputs:
IS_EXPENSIFY_EMPLOYEE: ${{ fromJSON(steps.checkAuthor.outputs.IS_EXPENSIFY_EMPLOYEE) }}

steps:
- name: Get merged pull request
id: getMergedPullRequest
uses: roryabraham/action-get-merged-pull-request@7a7a194f6ff8f3eef58c822083695a97314ebec1
with:
github_token: ${{ secrets.GITHUB_TOKEN }}

- name: Check whether the PR author is member of Expensify/expensify team
id: checkAuthor
run: |
if gh api /orgs/Expensify/teams/expensify-expensify/memberships/${{ steps.getMergedPullRequest.outputs.author }} --silent; then
echo "IS_EXPENSIFY_EMPLOYEE=true" >> "$GITHUB_OUTPUT"
else
echo "IS_EXPENSIFY_EMPLOYEE=false" >> "$GITHUB_OUTPUT"
fi
env:
GITHUB_TOKEN: ${{ secrets.OS_BOTIFY_TOKEN }}

newContributorWelcomeMessage:
runs-on: ubuntu-latest
needs: isExpensifyEmployee
if: ${{ github.actor != 'OSBotify' && !fromJSON(needs.isExpensifyEmployee.outputs.IS_EXPENSIFY_EMPLOYEE) && github.actor != 'imgbot[bot]' }}
steps:
# Version: 2.3.4
- name: Checkout
uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8
with:
token: ${{ secrets.OS_BOTIFY_TOKEN }}

- name: Get merged pull request
id: getMergedPullRequest
# TODO: Point back action actions-ecosystem after https://github.com/actions-ecosystem/action-get-merged-pull-request/pull/223 is merged
uses: roryabraham/action-get-merged-pull-request@7a7a194f6ff8f3eef58c822083695a97314ebec1
with:
github_token: ${{ secrets.GITHUB_TOKEN }}

- name: Get PR count for ${{ steps.getMergedPullRequest.outputs.author }}
run: echo "PR_COUNT=$(gh pr list --author ${{ steps.getMergedPullRequest.outputs.author }} --state any | grep -c '')" >> "$GITHUB_ENV"
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

- name: Comment on ${{ steps.getMergedPullRequest.outputs.author }}\'s first pull request!
if: ${{ fromJSON(env.PR_COUNT) == 1 }}
uses: actions-ecosystem/action-create-comment@cd098164398331c50e7dfdd0dfa1b564a1873fac
with:
github_token: ${{ secrets.OS_BOTIFY_TOKEN }}
number: ${{ steps.getMergedPullRequest.outputs.number }}
body: |
@${{ steps.getMergedPullRequest.outputs.author }}, Great job getting your first Expensify/App pull request over the finish line! :tada:

I know there's a lot of information in our [contributing guidelines](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md), so here are some points to take note of :memo::

1. Now that your first PR has been merged, you can be hired for another issue. Once you've completed a few issues, you may be eligible to work on more than one job at a time.
2. Once your PR is deployed to our staging servers, it will undergo quality assurance (QA) testing. If we find that it doesn't work as expected or causes a regression, you'll be responsible for fixing it. Typically, we would revert this PR and give you another chance to create a similar PR without causing a regression.
3. Once your PR is deployed to _production_, we start a 7-day timer :alarm_clock:. After it has been on production for 7 days without causing any regressions, then we pay out the Upwork job. :moneybag:

So it might take a while before you're paid for your work, but we typically post multiple new jobs every day, so there's plenty of opportunity. I hope you've had a positive experience contributing to this repo! :blush:

e2ePerformanceTests:
needs: [chooseDeployActions]
if: ${{ needs.chooseDeployActions.outputs.SHOULD_DEPLOY }}
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/reassurePerformanceTests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ on:
pull_request:
types: [opened, synchronize]
branches-ignore: [staging, production]
paths-ignore: [docs/**, .github/**, contributingGuides/**, tests/**, workflow_tests/**, '**.md', '**.sh']

jobs:
perf-tests:
Expand Down
19 changes: 19 additions & 0 deletions .github/workflows/shellCheck.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
name: Lint shell code

on:
workflow_call:
pull_request:
types: [opened, synchronize]
branches-ignore: [staging, production]
paths: ['**.sh']

jobs:
lint:
if: ${{ github.actor != 'OSBotify' || github.event_name == 'workflow_call' }}
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v3

- name: Lint shell scripts with ShellCheck
run: npm run shellcheck
1 change: 1 addition & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ on:
pull_request:
types: [opened, synchronize]
branches-ignore: [staging, production]
paths: ['**.js', '**.ts', '**.tsx', '**.sh', 'package.json', 'package-lock.json']

jobs:
jest:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/typecheck.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ on:
pull_request:
types: [opened, synchronize]
branches-ignore: [staging, production]
paths: ['**.ts', '**.tsx', 'package.json', 'package-lock.json']

jobs:
typecheck:
Expand Down
55 changes: 55 additions & 0 deletions .github/workflows/welcome.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,55 @@
name: Post new contributor welcome message

on:
push:
branches: [main]

jobs:
newContributorWelcomeMessage:
runs-on: ubuntu-latest
if: ${{ github.actor != 'OSBotify' && github.actor != 'imgbot[bot]' }}
steps:
- name: Checkout
uses: actions/checkout@v3

- name: Get merged pull request
id: getMergedPullRequest
run: |
read -r number author < <(gh pr list --search ${{ github.sha }} --state merged --json 'number,author' | jq -r '.[0] | [.number, .author.login] | join(" ")')
echo "number=$number" >> "$GITHUB_OUTPUT"
echo "author=$author" >> "$GITHUB_OUTPUT"
env:
GITHUB_TOKEN: ${{ github.token }}

- name: Check whether the PR author is member of Expensify/expensify team
id: isExpensifyEmployee
run: |
if gh api /orgs/Expensify/teams/expensify-expensify/memberships/${{ steps.getMergedPullRequest.outputs.author }} --silent; then
echo "IS_EXPENSIFY_EMPLOYEE=true" >> "$GITHUB_OUTPUT"
else
echo "IS_EXPENSIFY_EMPLOYEE=false" >> "$GITHUB_OUTPUT"
fi
env:
GITHUB_TOKEN: ${{ secrets.OS_BOTIFY_TOKEN }}

- name: Get PR count for ${{ steps.getMergedPullRequest.outputs.author }}
id: getPRCount
run: echo "PR_COUNT=$(gh pr list --author ${{ steps.getMergedPullRequest.outputs.author }} --state any | grep -c '')" >> "$GITHUB_OUTPUT"
env:
GITHUB_TOKEN: ${{ github.token }}

- name: Comment on ${{ steps.getMergedPullRequest.outputs.author }}\'s first pull request!
if: ${{ fromJSON(steps.getPRCount.outputs.PR_COUNT) == 1 }}
run: |
gh pr comment ${{ steps.getMergedPullRequest.outputs.number }} --body \
"@${{ steps.getMergedPullRequest.outputs.author }}, Great job getting your first Expensify/App pull request over the finish line! :tada:

I know there's a lot of information in our [contributing guidelines](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md), so here are some points to take note of :memo::

1. Now that your first PR has been merged, you can be hired for another issue. Once you've completed a few issues, you may be eligible to work on more than one job at a time.
2. Once your PR is deployed to our staging servers, it will undergo quality assurance (QA) testing. If we find that it doesn't work as expected or causes a regression, you'll be responsible for fixing it. Typically, we would revert this PR and give you another chance to create a similar PR without causing a regression.
3. Once your PR is deployed to _production_, we start a 7-day timer :alarm_clock:. After it has been on production for 7 days without causing any regressions, then we pay out the Upwork job. :moneybag:

So it might take a while before you're paid for your work, but we typically post multiple new jobs every day, so there's plenty of opportunity. I hope you've had a positive experience contributing to this repo! :blush:"
env:
GITHUB_TOKEN: ${{ github.token }}
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 1001039106
versionName "1.3.91-6"
versionCode 1001039200
versionName "1.3.92-0"
}

flavorDimensions "default"
Expand Down
7 changes: 1 addition & 6 deletions assets/images/MCCGroupIcons/MCC-Airlines.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
7 changes: 1 addition & 6 deletions assets/images/MCCGroupIcons/MCC-Commuter.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
7 changes: 1 addition & 6 deletions assets/images/MCCGroupIcons/MCC-Goods.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
7 changes: 1 addition & 6 deletions assets/images/MCCGroupIcons/MCC-Hotel.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
8 changes: 1 addition & 7 deletions assets/images/MCCGroupIcons/MCC-Mail.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Loading