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

[BE] Session resource #154

Closed
wants to merge 10 commits into from
Closed

Conversation

Frodo2412
Copy link
Collaborator

@Frodo2412 Frodo2412 commented Oct 15, 2023

What && Why

Esto lo abri desde #146 asi que va a haber que hacerle rebase a develop cuando esto se mergee.

Agrega CRUD de sesion de estudio mediante resources :sessions.

Veo un monton de cambios de front tambien que deben tener que ver con eso.

How

  • Step 1
  • Step 2
  • ...

Disclaimers / Extra Comments

Links

How to Review

  • Review commit by commit recommended.
  • Review all at once recommended.

Output

Screenshots

Mr-T-Smyth and others added 9 commits October 14, 2023 20:32
* remove groups next api, move group methods into group service

* include optional Options object in every call

* rename to ApiOptions and mark parameters in Options as optionals

* add options on calls that used

* fix bad call
@Frodo2412 Frodo2412 marked this pull request as draft October 15, 2023 00:02
@juanarias93
Copy link
Collaborator

@Frodo2412 no te conviene cambiar la base branch a feature/session-entity-2?
Cuando ese #146 se mergee la branch cambia automáticamente a develop.

@Frodo2412
Copy link
Collaborator Author

Ah no la tenia esa. Si me re conviene jajajajajaja.

@Frodo2412 Frodo2412 changed the base branch from develop to feature/session-entity-2 October 15, 2023 00:27
@Frodo2412 Frodo2412 closed this Oct 15, 2023
@juanarias93 juanarias93 deleted the be/session-resource branch October 31, 2023 22:54
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.

5 participants