Add HTML URL to API Issues

[jolheiser]

After the recent refactor, webhooks are sending URLs pointing to the API rather than the instance.

This PR adds the HTML URL to API Issues. API PRs already have the HTML URL, so I assumed it was okay to add to Issues as well.

[codecov-io]

Codecov Report

Merging #9654 into master will decrease coverage by 0.02%.
The diff coverage is 29%.

@@            Coverage Diff             @@
##           master    #9654      +/-   ##
==========================================
- Coverage   42.19%   42.16%   -0.03%     
==========================================
  Files         583      583              
  Lines       77254    77360     +106     
==========================================
+ Hits        32594    32618      +24     
- Misses      40651    40728      +77     
- Partials     4009     4014       +5

Impacted Files	Coverage Δ
models/issue_tracked_time.go	62.84% <100%> (+1.05%)	⬆️
routers/api/v1/api.go	73.96% <100%> (ø)	⬆️
routers/api/v1/repo/issue_tracked_time.go	39.56% <13.86%> (-5.7%)	⬇️
routers/api/v1/utils/utils.go	56% <70%> (+56%)	⬆️
modules/queue/workerpool.go	37.76% <0%> (-3.44%)	⬇️
services/pull/temp_repo.go	31.62% <0%> (-2.57%)	⬇️
modules/process/manager.go	78.31% <0%> (+3.61%)	⬆️
models/unit.go	67.56% <0%> (+5.4%)	⬆️

---

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 da8a942...948c2a9. Read the comment docs.

[guillep2k]

Looks like the refactor introduced a breaking change. If that was not intentional, I'd rather add APIURL and use URL again for the HTML link.

[lafriks]

This is also consistent with what github returns

[jolheiser]

Looks like the refactor introduced a breaking change. If that was not intentional, I'd rather add APIURL and use URL again for the HTML link.

This is consistent with how structs/pull.go does it as well.
Since the structs package is generally used by APIs, I think the thought is that URL refers to the APIs URL?

[guillep2k]

This is consistent with how structs/pull.go does it as well.
Since the structs package is generally used by APIs, I think the thought is that URL refers to the APIs URL?

No problem, I thought it could be a breaking change, that's all.

[jolheiser]

This will also need a backport FYI.

[jolheiser]

Will backport in a bit.