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.

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

Vebt 887 - Search by program #1276

Merged
merged 5 commits into from
Dec 31, 2024
Merged

Vebt 887 - Search by program #1276

merged 5 commits into from
Dec 31, 2024

Conversation

alexchan-va
Copy link
Contributor

@alexchan-va alexchan-va commented Dec 30, 2024

Description

Add to Comparison Tool ability to search by program. The endpoint is called program and is hit when a program description and coordinates are sent.

Original issue(s)

877

As a...

VEBT Developer

I want…

To build the Search By Name section of the CT redesign

So that…

CT redesign work can be completed

Acceptance Criteria

User must search for a program AND location
School, employer or training program & city, state or postal code are both *REQUIRED fields.
Search by program mimic search by name.
Back end to do both name and location as part of code
Technical Details:

Dependencies:

Collaborate with EDM to utilize one feature toggle
BE will need the URL parameters for the data call
EDM will need to provide the program data via excel doc in GIDS
Feature Flag needed: Yes

Definition of done

  • [x ] Events are logged appropriately
  • [x ] Documentation has been updated, if applicable
  • [ x] No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs

@alexchan-va alexchan-va requested a review from a team as a code owner December 30, 2024 20:13
@alexchan-va alexchan-va merged commit 38144e6 into master Dec 31, 2024
4 checks passed
@alexchan-va alexchan-va deleted the vebt-887 branch December 31, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants