Skip to content

Commit

Permalink
fix: fix some pr issues
Browse files Browse the repository at this point in the history
  • Loading branch information
chadbrokaw committed Jan 24, 2025
1 parent 3d52243 commit 8ef2fac
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion app/assets/javascripts/account/AccountService.js.coffee
Original file line number Diff line number Diff line change
Expand Up @@ -91,12 +91,13 @@ AccountService = (
Service._reformatDOB()
return true
).catch((response) ->
# for errors we manually stop the loading overlay

AnalyticsService.trackEvent("login_failed", {
user_id: null,
origin: 'Application Sign In',
error_reason: response?.reason || undefined,
})
# for errors we manually stop the loading overlay
bsLoadingOverlayService.stop()
return false
)
Expand Down
2 changes: 1 addition & 1 deletion app/javascript/__tests__/hooks/useGTMDataLayer.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ describe("useGTMDataLayer", () => {
expect(consoleSpy).toHaveBeenCalled()
})

it("errors out when no event is provided in the data object", () => {
it("errors out when an event property is provided in the data object", () => {
const event = "testEvent"
const data = { test: "data", event: "testEvent" }

Expand Down

0 comments on commit 8ef2fac

Please sign in to comment.