Skip to content

Clarify events backend project scope and cart design expectations#295

Merged
adamblanchard merged 6 commits intomainfrom
backedn-mid-specialism-projects-improvements-from-feedback
Apr 16, 2026
Merged

Clarify events backend project scope and cart design expectations#295
adamblanchard merged 6 commits intomainfrom
backedn-mid-specialism-projects-improvements-from-feedback

Conversation

@oleksiybondar
Copy link
Copy Markdown
Contributor

Updates the events-startup-project docs to remove ambiguity around MVP scope, API expectations, and database design.
Key clarifications include:

  • required vs optional Week 5 backend features
  • reserved-word naming guidance for schema design
  • backend-persisted carts for both guest and authenticated users
  • one-active-cart expectations and nullable cart.user_id hint
  • default cart-line route/key design, with composite-key design noted as an alternative
  • search via query parameters on GET /api/events
  • explicit note that inventory is intentionally unlimited for this project

Copy link
Copy Markdown
Contributor

@adamblanchard adamblanchard left a comment

Choose a reason for hiding this comment

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

thanks for the quick changes and clarifications! Overall i like the idea of this section, also for future use to add tips as we discover new things that we can guide trainees further on.

My main suggestion is to remove the scope/mvp section and update other docs first.

Comment thread courses/backend/events-startup-project/requirements.md Outdated
Comment thread courses/backend/events-startup-project/requirements.md Outdated

These clarifications define the minimum expectations for this project. They are intended to remove ambiguity while still leaving room for implementation choices.

### Scope and MVP
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This scope section looks like it's repeating some things already written elsewhere (e.g. admin optional scope is already written above).

Is there anything in here that's not already described above or in the main project readme?

If so, i suggest we update the main readme (for deliverables) and the requirements above (for scope) rather than create another list that trainees need to check (mostly) the same things.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I was trying to solves some of Alis unclarities, but yeah its written elsewhere, so we can remove it


We already have a very basic API in place, but it is missing many features that you will be required to implement. You can find the current API in the project template.

## Implementation Clarifications
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like the idea of this "tips" section for some extra help :)

Comment thread courses/backend/events-startup-project/weekly-plan.md Outdated
oleksiybondar and others added 4 commits April 16, 2026 11:53
Co-authored-by: Adam Blanchard <adamgoodie@gmail.com>
Co-authored-by: Adam Blanchard <adamgoodie@gmail.com>
Co-authored-by: Adam Blanchard <adamgoodie@gmail.com>
@adamblanchard adamblanchard merged commit 2611d6e into main Apr 16, 2026
3 checks passed
@adamblanchard adamblanchard deleted the backedn-mid-specialism-projects-improvements-from-feedback branch April 16, 2026 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants