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

- fix: several fixes while testing whole workflow #48

Merged
merged 16 commits into from
Oct 18, 2023
Merged

- fix: several fixes while testing whole workflow #48

merged 16 commits into from
Oct 18, 2023

Conversation

agallardol
Copy link
Collaborator

@agallardol agallardol commented Oct 17, 2023 •
edited
Loading

Fixes:

  • i18n labels
  • Missing titles on most pages
  • Error sending message to inbox
  • UI text alignment
  • Navbar missalignment when back arrow is hidden

Improvements:

  • Added auto focus to form inputs
  • Removed unused code
  • Added refetch logic for messages (hardcoded until we implement a socket... currently refetch every 5 seconds)
  • Added missing agents message

@agallardol agallardol marked this pull request as ready for review October 17, 2023 19:40
@agallardol agallardol merged commit f118db3 into main Oct 18, 2023
4 checks passed
@nicarq nicarq deleted the agallardol/issues-fix branch October 18, 2023 12:44
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
@agallardol