Move notification APIFormat

[6543]

as title

[codecov-io]

Codecov Report

Merging #13783 (724aaab) into master (253add8) will decrease coverage by 0.00%.
The diff coverage is 63.41%.

@@            Coverage Diff             @@
##           master   #13783      +/-   ##
==========================================
- Coverage   42.23%   42.22%   -0.01%     
==========================================
  Files         699      700       +1     
  Lines       76959    76959              
==========================================
- Hits        32500    32496       -4     
- Misses      39094    39097       +3     
- Partials     5365     5366       +1     

Impacted Files	Coverage Δ
models/notification.go	66.25% <ø> (+0.49%)	⬆️
modules/convert/notification.go	60.52% <60.52%> (ø)
routers/api/v1/notify/repo.go	72.97% <100.00%> (ø)
routers/api/v1/notify/threads.go	56.25% <100.00%> (ø)
routers/api/v1/notify/user.go	25.00% <100.00%> (ø)
modules/charset/charset.go	68.53% <0.00%> (-4.50%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 253add8...724aaab. Read the comment docs.

[gary-kim]

LGTM

[zeripath]

I'm still not certain about whether convert is the nicest design - I guess it's one way of coming up with an exportable set of models even if we don't use them internally.

However, this PR makes notification consistent with the rest of the library, therefore LGTM.