Skip to content
This repository has been archived by the owner on Apr 3, 2023. It is now read-only.

Feat/users advanced search #36

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

Conversation

TeddyRoncin
Copy link
Member

@TeddyRoncin TeddyRoncin commented Feb 3, 2023

Description

Checklist

Test

Implementation

  • Files and variables have a name explaining what they do.
  • Updating the DB schemas and seeding the tables do not generates errors.
  • All code repeted more than 2 times has been refactored in a dedicated file or function.
  • Debugs have been removed : dd() | dump()

Tools

  • PHP CS Fixer has fixed all files inside the src folder.
  • All commits have been done using Commitizen.

Documentation

  • All new classes and non-trivial functions have a code documentation.
  • There are comments to explain complex parts inside your code.
  • If a new folder is created, it has been added and explained inside the "Folder structure" part of the README.md file.

Teddy Roncin added 2 commits February 2, 2023 16:25
Added many filters for the /users routes. User's nickname are now returned when fetching a collection of users
…e current version

Some code (in the filters) was outdated and not updated during the rebase
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant