Skip to content

Draft: Refactor error tracking gitlab details

Dmytro Zaporozhets (DZ) requested to merge dz-error-tracking-gitlab-detail-refactoring into master

What does this MR do and why?

Move error.gitlab_commit and error.gitlab_commit_path attributes assignment from settings class to service.

  1. This way we can re-use this logic for integrated error tracking in follow-up mrs
  2. Settings class should be responsible only for settings. Instead we call it from services when query errors list, error details, update error etc. We should move such behavior away from settings.

Screenshots or screen recordings

No changes in UI

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Dmytro Zaporozhets (DZ)

Merge request reports