Skip to content

Qb auth frontend#190

Merged
jacktoncelli merged 12 commits intomainfrom
qb-auth-frontend
Nov 29, 2025
Merged

Qb auth frontend#190
jacktoncelli merged 12 commits intomainfrom
qb-auth-frontend

Conversation

@abby-ux
Copy link
Copy Markdown
Contributor

@abby-ux abby-ux commented Nov 25, 2025

Description

[Link to Ticket](insert the link to your ticket inside the parenthesis here)

Please include a summary of the changes. If there were unexpected changes
to separate processes/components/dependencies, please explain why. Bullet list is fine!

Backend PRs:

  • I updated relevant API documentation and tests

Frontend PRs:

Page route(and description of how to get desired flow if necessary)

Please include any relevant images of the changes here.

Did any new/other issues pop up?

No

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • New and existing unit tests pass locally with my changes

Copy link
Copy Markdown
Contributor

@jacktoncelli jacktoncelli left a comment

Choose a reason for hiding this comment

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

As I said in dms, I am having issues with consistently creating a companyExternal object. Once I got one created (I'm not sure how tbh) it the redirect worked and stuff and otherwise it seems to work in terms of getting the right URLs and returning to frontend.

I pushed some code that fixes import on login

Copy link
Copy Markdown
Contributor

@jacktoncelli jacktoncelli left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for debugging

For future reference - Quickbooks only allows one authenticated connection from Prisere per quickbooks company, which was causing the problems we saw before.

@jacktoncelli jacktoncelli merged commit d71e4f9 into main Nov 29, 2025
3 checks passed
@jacktoncelli jacktoncelli deleted the qb-auth-frontend branch November 29, 2025 22:50
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.

2 participants