Updated hasPermission here: https://phab.comm.dev/D9844
- Feed Queries
- All Stories
- Search
- Feed Search
- Transactions
- Transaction Logs
Nov 13 2023
In D9841#287271, @ashoat wrote:If this is still relevant, should I update hasPermission to always ensure that KNOW_OF bit is set?
Yes, absolutely. The default should definitely be "match existing code" – if you ever have a question like this in the future for this project, please default to matching existing code, instead of doing things differently and hoping your reviewer will notice a question in the diff summary.
Actually, just make sure it's fixed before landing. Don't think it's worth requesting changes
Requesting changes because I think HLOCALSTACK_HOST should be LOCALSTACK_HOST. Feel free to re-request if I'm missing something
Nov 11 2023
remove comments
Nov 10 2023
Resolved by D9832
Nov 9 2023
Created a Linear issue here: https://linear.app/comm/issue/ENG-5692/jest-issue-with-derived-tshape-validators
reposition spreads to top
Nov 8 2023
Probably missing something here.
land
Nov 7 2023
land
improve readability by using lodash mapValues
land
Nov 6 2023
fix test
land
land
rebase and land
land
update regex from {1,16} to +
update tHexEncodedRolePermission to accomodate larger strings
fix regex boundary and add failing test
add failing hello a02 test case
update regex to use ^ and $ instead of \b
rebase and land