Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: getlago/lago-front
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v1.33.4
Choose a base ref
...
head repository: getlago/lago-front
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v1.34.0
Choose a head ref
  • 12 commits
  • 56 files changed
  • 4 contributors

Commits on Sep 18, 2025

  1. misc: make invoice related permissions relies on the same logic (#2462)

    Taking the opportunity to remove the `TODO: Compare this with
    src/hooks/usePermissionsInvoiceActions.ts:` comments in the app.
    
    I've come to the conclusion that `canFinalize` should check for an
    additional permission so i changed it and adapted the related test
    ansmonjol authored Sep 18, 2025
    Configuration menu
    Copy the full SHA
    7e137b3 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    514ff49 View commit details
    Browse the repository at this point in the history
  3. misc: fetch fees and invoicesubscriptions independently of the invoic…

    …e object details (#2459)
    
    ## Context
    
    We're often facing a `maximum-complexity` graphql error when adding more
    data to fetch within the invoice details view.
    
    This view works like a huge get, that retrieve all the data at once.
    The query became that big that we reached the default complexity limit
    (365) and are blocked when needing to extend this page's features.
    
    This PR attempt to fix that by splitting the query into smaller ones.
    This improvement does not aims to fix the fragment architecture spread
    over all components, unless they affect directly this query.
    
    Note that we cannot not use the same endpoint within the same query
    using aliases, as they would still have impact on the same "final" query
    complexity.
    
    ## Description
    
    The 2 commit changes aims to separate the invoice informations, invoice
    `fees` and invoice `invoiceSubscriptions` into 3 separates queries.
    - invoice informations: stays into the `getInvoiceDetails`
    - invoice `fees`: new `query getInvoiceFees`
    - invoice `invoiceSubscriptions`: new `query getInvoiceSubscriptions`
    
    Having them separated and passed as different props allows us to define
    specific types for each of them, defined in
    `src/components/invoices/types.ts`.
    
    Some `refetchQueries` had to be updated to fetch the correct query
    impacting the correct part of the invoice (here related to
    `invoiceSubscriptions`)
    
    Some type manual casting could have been removed along the way.
    
    I performed a manual QA on those scenario and they all work for the
    different kind of invoices we have (sub, one-off, credit)
    - edit fee
    - regenerate invoice
    - update invoice statuses (payment, ...)
    
    <!-- Linear link -->
    Fixes ISSUE-1146
    ansmonjol authored Sep 18, 2025
    Configuration menu
    Copy the full SHA
    fd3f7c1 View commit details
    Browse the repository at this point in the history
  4. deps: bump cypress version to latest (#2464)

    We have a [low severity
    CVE](https://github.com/getlago/lago-front/security/dependabot/113) on a
    sub deps of cypress.
    
    It's fixed in their `15.0.0` so let's upgrade to the latest.
    
    This `v15` brings the cypress studio and would be great to test it soon!
    
    Other packages needs to be updated to have the CVE totally fixed on our
    repo but they are not there yet.
    ansmonjol authored Sep 18, 2025
    Configuration menu
    Copy the full SHA
    62ed0a3 View commit details
    Browse the repository at this point in the history
  5. chore: set correct name for the subscription fee section (#2465)

    Simple renaming 
    
    I'm working on a new feature that will add fixed fee on the plan.
    
    The current section related to the Subscription fee is named "Fixed fee"
    and better to be renamed to prevent conflicts.
    ansmonjol authored Sep 18, 2025
    Configuration menu
    Copy the full SHA
    63b46c7 View commit details
    Browse the repository at this point in the history

Commits on Sep 19, 2025

  1. feat(invoice): display wallet or transaction name on fee line (#2463)

    ## Context
    
    We recently added the possibility to name a wallet transaction.
    At the same time, the wallet name was never displayed in the invoice
    details at fee level.
    
    ## Description
    
    This PR fixes and improve the wallet related fee
    - If the wallet name is present, it becomes the fee display name
    - if the wallet transaction name is prevent, we concat it with a
    "default" string
    - Otherwise we define this "default" string only
    ansmonjol authored Sep 19, 2025
    Configuration menu
    Copy the full SHA
    3affe99 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    9be19f0 View commit details
    Browse the repository at this point in the history
  3. [ISSUE-1148] Prevent cached usage data (#2467)

    ## Context
    
    The usage query was caching the first values received, making the drawer
    data innacurate.
    
    ## Description
    
    Prevent caching of the usage query.
    
    <!-- Linear link -->
    Fixes ISSUE-1148
    stephenlago99 authored Sep 19, 2025
    Configuration menu
    Copy the full SHA
    c3f8b7f View commit details
    Browse the repository at this point in the history
  4. misc: rename usage charge related elements (#2468)

    Simple renaming to better match usage and prepare future implementations
    ansmonjol authored Sep 19, 2025
    Configuration menu
    Copy the full SHA
    dac3dd4 View commit details
    Browse the repository at this point in the history
  5. feat(ai-agent): support graphql subscriptions (#2448)

    ## Context
    
    Related to ai-agent feature.
    
    This PR should update the instantiation of graphql to handle
    subscriptions.
    
    ## Description
    
    I added actioncable because that's the protocol we're using on the
    backend side.
    I've also split the `init` file to extract links for readability. 
    
    As subscriptions are not used in the app yet, this can be safely merged
    keellyp authored Sep 19, 2025
    Configuration menu
    Copy the full SHA
    1fdf60d View commit details
    Browse the repository at this point in the history

Commits on Sep 22, 2025

  1. misc: compose charge accordion elements for better reusability (#2469)

    ## Context
    
    I'm planning to have a whole new Charge accordion for fixed charges.
    
    Those will have some differences with the event-based charges and today,
    the only way to implement those change is either by making the charge
    accordion API worst or make inner elements to be more "composable".
    
    ## Description
    
    This PR starts to "compose" some components of the charge accordion, so
    we can create this fixed charges accordion easier later.
    The "main" benefit is visible in
    `src/components/plans/UsageChargeAccordion.tsx` and in some create form
    pages.
    
    I've created a commit per split, and I'll create other PR as here I
    think I only split half if the component.
    
    It's a ready a again in readability and size (1116 lines to 739).
    ansmonjol authored Sep 22, 2025
    Configuration menu
    Copy the full SHA
    2aaad58 View commit details
    Browse the repository at this point in the history

Commits on Sep 23, 2025

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