Skip to content
This repository was archived by the owner on Sep 26, 2025. It is now read-only.

fix: update schema#956

Merged
dbm03 merged 1 commit into
mainfrom
fix/graphql-middleware-accounting-vendor
Apr 24, 2025
Merged

fix: update schema#956
dbm03 merged 1 commit into
mainfrom
fix/graphql-middleware-accounting-vendor

Conversation

@dbm03
Copy link
Copy Markdown
Member

@dbm03 dbm03 commented Apr 24, 2025

User description

Update vendor code to fix the internal server error with accounting.go middleware


PR Type

Bug fix


Description

  • Update github.com/nhost/be dependency to fix accounting middleware error

  • Resolve internal server error in accounting.go middleware


Changes walkthrough 📝

Relevant files
Dependencies
go.mod
Update nhost/be dependency version                                             

go.mod

  • Update github.com/nhost/be dependency to version
    20250424140218-1be7f62eade6
  • +1/-1     
    go.sum
    Update go.sum with new nhost/be dependency                             

    go.sum

  • Remove old github.com/nhost/be dependency entry
  • Add new github.com/nhost/be dependency entry with updated version
  • +2/-0     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link
    Copy Markdown
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @github-actions
    Copy link
    Copy Markdown
    Contributor

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @dbarrosop dbarrosop changed the title fix: graphql middleware accounting vendor fix: update schema Apr 24, 2025
    @dbm03 dbm03 merged commit b17bbd6 into main Apr 24, 2025
    10 checks passed
    @dbm03 dbm03 deleted the fix/graphql-middleware-accounting-vendor branch April 24, 2025 16:33
    Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    2 participants