fix(orm)!: timestamp encoding doesn't handle nil values properly

[aaronc]

Description

Ref #11980

---

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

• included the correct type prefix in the PR title
• added ! to the type prefix if API or client breaking change
• targeted the correct branch (see PR Targeting)
• provided a link to the relevant issue or specification
• followed the guidelines for building modules
• included the necessary unit and integration tests
• added a changelog entry to CHANGELOG.md
• included comments for documenting Go code
• updated the relevant documentation or specification
• reviewed "Files changed" and left comments if necessary
• confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.

I have...

• confirmed the correct type prefix in the PR title
• confirmed ! in the type prefix if API or client breaking change
• confirmed all author checklist items have been addressed
• reviewed state machine logic
• reviewed API design and naming
• reviewed documentation is accurate
• reviewed tests and test coverage
• manually tested (if applicable)

[codecov]

Codecov Report

Merging #12273 (40167f0) into main (2739f83) will increase coverage by 9.85%.
The diff coverage is n/a.

❗ Current head 40167f0 differs from pull request most recent head b04a9ca. Consider uploading reports for the commit b04a9ca to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #12273      +/-   ##
==========================================
+ Coverage   56.25%   66.11%   +9.85%     
==========================================
  Files         667      675       +8     
  Lines       56576    71275   +14699     
==========================================
+ Hits        31829    47122   +15293     
+ Misses      22165    21508     -657     
- Partials     2582     2645      +63     

Impacted Files	Coverage Δ
x/auth/client/cli/validate_sigs.go	0.00% <0.00%> (-80.00%)	⬇️
x/genutil/client/cli/validate_genesis.go	0.00% <0.00%> (-78.05%)	⬇️
x/auth/client/cli/broadcast.go	0.00% <0.00%> (-75.00%)	⬇️
x/distribution/client/cli/query.go	0.00% <0.00%> (-73.95%)	⬇️
x/slashing/keeper/msg_server.go	9.09% <0.00%> (-73.52%)	⬇️
x/distribution/client/cli/tx.go	4.13% <0.00%> (-71.15%)	⬇️
x/genutil/client/cli/migrate.go	0.00% <0.00%> (-65.34%)	⬇️
x/staking/client/cli/query.go	0.00% <0.00%> (-63.07%)	⬇️
x/gov/client/cli/tx.go	14.00% <0.00%> (-56.52%)	⬇️
x/bank/keeper/msg_server.go	3.03% <0.00%> (-51.77%)	⬇️
... and 561 more

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[tac0turtle]

closing as orm is not an epic within the current workscope of the team. We can reopen when we complete app wiring

[aaronc]

Reopening this as I'd like to quickly address the outstanding issuing blocking ORM from being considered v1.0 and feature complete.

[julienrbrt]

I think we should have docs, upgrading tooling on that. Otherwise, it feels a bit like "good luck".

[aaronc]

I'm planning to push another PR for ORM migrations that will automate this

[julienrbrt]

That queue thing does not work.

[aaronc]

That queue thing does not work.

What's the queue?

[ryanchristo]

ACK