Skip to content

Add event detail - eventType to 'remote-input-success' customEvent #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Apr 8, 2024
Merged

Add event detail - eventType to 'remote-input-success' customEvent #34

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
5fb21bb
Add event detail - eventType to success customEvent
kendallgassner Mar 28, 2024
7886c72
clean;
kendallgassner Mar 28, 2024
a0aab0e
fix debounce
kendallgassner Mar 29, 2024
92337bc
Add instanceof to error
TylerJDev Mar 29, 2024
4a212b1
Update test
TylerJDev Mar 29, 2024
a2157a3
Merge branch 'kendallg/add-event-detail-eventType' of https://github.…
TylerJDev Mar 29, 2024
07422fd
formatting
TylerJDev Mar 29, 2024
af8252a
add back eslint
TylerJDev Mar 29, 2024
5c96242
remove lint for now -- will add updated lint versions in a new PR
TylerJDev Mar 29, 2024
e120777
Apply suggestion from @kendallgassner
kendallgassner Mar 29, 2024
7fdbaad
Apply suggestions from code review
kendallgassner Mar 29, 2024
b5e7c4f
formatting
TylerJDev Mar 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions .github/workflows/pr.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
name: Build and run tests

'on':
push:
branches:
- main
pull_request:
types:
- opened
- synchronize

jobs:
test:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Install dependencies
run: npm ci
- name: Run build
run: npm run build
- name: Run test
run: npm run test
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can't believe we weren't doing this already 🤦🏻

Loading