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.

Sign up for GitHub

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

Jump to bottom

Simplify usage of authenticatedData and some do some code clean-up #1015

Open
wants to merge 4 commits into
base: features/improve-agent-management
Choose a base branch
from
Open

Simplify usage of authenticatedData and some do some code clean-up #1015

wants to merge 4 commits into from
+162 −341

Conversation

sabaimran
Copy link
Member

  • De facto, was being assumed everywhere if authenticatedData is null, that it's not logged in. This isn't true because the data can still be loading. Update the hook to send additional states.
  • Bonus: Delete model picker code and a slew of unused imports.

@sabaimran
Clean up all references to authenticatedData
e9dae42 
- De facto, was being assumed everywhere if authenticatedData is null, that it's not logged in. This isn't true because the data can still be loading. Update the hook to send additional states.
- Bonus: Delete model picker code and a slew of unused imports.
@sabaimran
Fix conditional reference to is mobile width hook
2c7c16d
@sabaimran
Fix handling 403 forbidden error from auth response
a17cc9d
@sabaimran
Set width of chat layout to 100%
8c6b421
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@sabaimran