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

Feat/ue routes #35

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

Feat/ue routes #35

wants to merge 3 commits into from

Conversation

TeddyRoncin
Copy link
Member

Description

Stuff about UEs

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 January 23, 2023 11:23
…breaking changes were fixed

Most notable changes are the change of the version of Symfony, ApiPlatform and PHP
Added route GET /ues/search : it can search UEs by their code and the type of ECTS they give. Added route GET /ues/{code} that returns informations about the UE with the provided code. Added a new search
filter for ApiPlatform which allows to change the name of the parameters in the URL. Fixed a small bug in the UpdateAtSubscriber class that could make the server produce an error
…g : filiere not displayed

Fix needed : display filiere on Get route (user:read:one)
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