Skip to content
This repository has been archived by the owner on Aug 16, 2024. It is now read-only.

fix: improve TOPS report error handling #54

Merged
merged 1 commit into from
Feb 8, 2024
Merged

fix: improve TOPS report error handling #54

merged 1 commit into from
Feb 8, 2024

Conversation

fibble
Copy link
Contributor

@fibble fibble commented Feb 7, 2024 •
edited
Loading

Description

Improve TOPS Report Error handling

Related issue: VOL-4969

Before submitting (or marking as "ready for review")

  • Does the pull request title follow the conventional commit specification?
  • Have you performed a self-review of the code
  • Have you have added tests that prove the fix or feature is effective and working
  • Did you make sure to update any documentation relating to this change?

@JoshuaLicense JoshuaLicense merged commit 45bb578 into main Feb 8, 2024
3 of 6 checks passed
@JoshuaLicense JoshuaLicense deleted the improve-tops-report-link-error-logging branch February 8, 2024 16:32
@github-actions github-actions bot mentioned this pull request Feb 8, 2024
Open
jerotire pushed a commit to dvsa/vol-app that referenced this pull request Aug 16, 2024
@fibble
fix: improve TOPS report error handling (dvsa/olcs-selfserve#54)
7766ca4
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@JoshuaLicense JoshuaLicense JoshuaLicense approved these changes

@ilindsay ilindsay ilindsay approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fibble @JoshuaLicense @ilindsay