Skip to content
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

Jump to bottom

Auto-fills new department from Firecares ID #497

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Auto-fills new department from Firecares ID #497

wants to merge 2 commits into from

Conversation

bliitzkrieg
Copy link
Contributor

Overview

Added logic to auto fill the department details when Firecares ID is entered.

GitHub Issues

Changes

  • Added endpoint to query firecares
  • Added onBlur function to Firecares ID input

Screenshots / Videos

Screen Shot 2020-06-17 at 7 16 02 PM

Steps to Test

  • Login to admin account
  • Navigate to /admin/fireDepartment/new
  • Enter Firecares ID and blur input

@bliitzkrieg bliitzkrieg requested a review from chopchop505 June 17, 2020 23:20
@bliitzkrieg bliitzkrieg self-assigned this Jun 17, 2020
chopchop505
chopchop505 requested changes Jun 18, 2020
View reviewed changes
server/api/fire-department/index.js Outdated Show resolved Hide resolved
@bliitzkrieg bliitzkrieg requested a review from chopchop505 June 19, 2020 00:59
@bliitzkrieg
Copy link
Contributor Author

This is ready for review/merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chopchop505 chopchop505 Awaiting requested review from chopchop505

Assignees

@bliitzkrieg bliitzkrieg

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bliitzkrieg @chopchop505