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

Api/fixing the filters #198

Merged
merged 2 commits into from
Feb 1, 2024
Merged

Api/fixing the filters #198

merged 2 commits into from
Feb 1, 2024

Conversation

TheSloanRanger
Copy link
Collaborator

@TheSloanRanger TheSloanRanger commented Feb 1, 2024


name: API/fixing the filters

Pull Request Overview

Brief Description:

fixing the filters within the API

Issue Number:

n/a

Related User Story:

#6

Code Changes

Major Changes:

fixed the filtering

Minor Changes:

n/a

Testing

Test Cases Covered:

n/a

Documentation

Updated Documentation:

n/a

Review Checklist

  • Code follows project's coding standards.
  • Code is commented and clean.
  • Tests are comprehensive and passing.
  • Documentation is updated and accurate.

@TheSloanRanger TheSloanRanger added the API An API task label Feb 1, 2024
@TheSloanRanger TheSloanRanger added this to the API milestone Feb 1, 2024
@TheSloanRanger TheSloanRanger self-assigned this Feb 1, 2024
@TheSloanRanger TheSloanRanger reopened this Feb 1, 2024
Copy link
Collaborator

@FinnbarHome FinnbarHome left a comment

Choose a reason for hiding this comment

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

LGTM but Adam will have to force merge this

@FinnbarHome FinnbarHome merged commit 5b2f9e1 into dev Feb 1, 2024
6 of 9 checks passed
@FinnbarHome FinnbarHome deleted the API/fixing-the-filters branch February 1, 2024 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API An API task
Projects
Development

Successfully merging this pull request may close these issues.

2 participants