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

refactor(ui): perks section and tickets page #566

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

Commits on Dec 12, 2023

  1. refactor(ui): perks section and tickets page

    - Converted `BaristaPerksSection` to a `StatelessWidget` as it did not utilise any stateful features.
    - Transitioned `BaristaPerksSection` and `UserRoleIndicator` to accept a `String` instead of `Role`. This prevents duplicate conversion/formatting from `Role` to `String` in these widgets.
    - Made `tickets_page.dart` more state safe by removing UserState cast and taking into account cases where UserState is not UserWithData.
    - Extracted ticket contents of `TicketsPage` to a private widget to maintain readablity.
    - Fixed the incorrect usage of `context.read` inside of the build method of `TicketsPage`.
    
    The refactor improves overall code maintainability and aligns with best practices for widgets that do not require internal state management.
    marfavi committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    8eea334 View commit details
    Browse the repository at this point in the history

Commits on Jan 29, 2024

  1. Configuration menu
    Copy the full SHA
    7b5e70d View commit details
    Browse the repository at this point in the history