Labels Milestones

error-logging #29

Merged

trentlarson merged 12 commits from error-logging into master 2 years ago

Conversation 2 Commits 12 Files Changed 13

anomalist commented 2 years ago

Owner

Added an AlertMessage component and did some overall cleanup. Needs testing ... how can we trigger errors here?

Added an AlertMessage component and did some overall cleanup. Needs testing ... how can we trigger errors here?

anomalist added 9 commits 2 years ago

b139957e3e Added identity check

3f6dbdebef Moved checkLimits inside try...catch since it has an identity check

99ea161da0 Removed created() and relocated to mounted() could not replicate issue

bcc6de6fc0 New AlertMessage component

8d8635a3e6 Added try...catch and linted

3c977a1f28 First roll out of an AlertMessage control

0af03227a6 Propagated AlertMessage component

c31445865e Straggler

0fd4b86a84 A bit of linting

trentlarson was assigned by anomalist 2 years ago

trentlarson commented 2 years ago

Owner

On many pages, one way to test is to disconnect from a network and try some actions. (That's the majority... so if those work then I'd feel safe.)

On many pages, one way to test is to disconnect from a network and try some actions. (That's the majority... so if those work then I'd feel safe.)

anomalist commented 2 years ago

Poster

Owner

On many pages, one way to test is to disconnect from a network and try some actions. (That's the majority... so if those work then I'd feel safe.)

That makes sense. I guess I can set a browser to Presets | Offline.

> On many pages, one way to test is to disconnect from a network and try some actions. (That's the majority... so if those work then I'd feel safe.) That makes sense. I guess I can set a browser to Presets | Offline.

anomalist added 1 commit 2 years ago

c24022c41c Make the visibility of the Alert indirect

anomalist added 1 commit 2 years ago

c859778832 Add AlertMessage component

anomalist added 1 commit 2 years ago

0a020a4069 Clean up vestigle old alert code

trentlarson merged commit d98e95915b into master 2 years ago

trentlarson referenced this issue from a commit 2 years ago

Merge pull request 'error-logging' (#29) from error-logging into master

The pull request has been merged as d98e95915b.

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels

No items

No Label

Milestone

Set milestone

Clear milestone

No items

No Milestone

Assignees

Assign users

Clear assignees

No Assignees

trentlarson

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview

Loading…

Cancel

Save

There is no content yet.