- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Jul 18 2024
Landed this so we could include it in the next release
feedback
In D12800#362553, @ashoat wrote:Great work on this – looks really elegant!
Where did we land the MariaDB params – can those be specified only in the .env file, or will they need to be additionally specified to Terraform somehow?
Great work on this – looks really elegant!
Here's the followup diff I promised: D12799
land
rebase before landing
rebase BEFORE re-removing any-cast
cherry-pick and land
cherry-pick and land
It's actually fine to land this as-is. If migration runs on RawThreadInfo that's already missing permissions, nothing will happen since stripPermissionsFromMemberInfo just won't really do anything:
rebase
rebase
rebase
rebase
rebase
rebase
rebase
Can drop this now, thanks @marcin !
cherry-pick and land
cherry-pick and land
cherry-pick and land
This is purely rename diff, so safe to cherry-pick and land out of order.
Did you perhaps forget to remove the any-cast before diffing this up?
remove extraneous any-cast
commandeer
bartek is afk so i'm going to commandeer and land
Thanks for all of the explanations above, those all make sense. @tomek and I discussed this in our 1:1 today and settled on keeping this approach, at least for now.
Looks like D12784 has been landed, so this can be abandoned after you rebase
Thank you!!
Rename fix
Shorthand
Update types and names
Rename