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.

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

Fixes #358

Merged
merged 20 commits into from
May 31, 2024
Merged

Fixes #358

merged 20 commits into from
May 31, 2024

Conversation

mowolf
Copy link
Member

@mowolf mowolf commented Jan 16, 2024

Klappt das?

@mowolf
Copy link
Member Author

mowolf commented Jan 16, 2024

bei mir liefs kurz, jetzt geht nix mehr...

@mowolf
Copy link
Member Author

mowolf commented Jan 16, 2024

bei mir läufts mit node 18

@mowolf
Copy link
Member Author

mowolf commented Jan 16, 2024

  • Bild wird auf iOS generiert bei großem chat

Copy link

github-actions bot commented Jan 16, 2024

Visit the preview URL for this PR (updated for commit b20ebcc):

https://whatsanalyze-80665--pr358-fixes-t2sgj95h.web.app

(expires Fri, 07 Jun 2024 07:36:19 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 5796a863d3a1b99df3fcb31feec7a39514d97aa4

@mowolf
Copy link
Member Author

mowolf commented Jan 16, 2024

image

Prozess bar for pdf is now really reflecting how much is processed

@mowolf
Copy link
Member Author

mowolf commented Jan 16, 2024

on iOS large chat pdf generation crashes and reloads page without error, probably memory issue

Copy link
Member

@fellnerse fellnerse left a comment

Choose a reason for hiding this comment

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

wenn du die node version anheben willst, dann musst du das auch an allen stellen machen:


und allen anderen pipelines auch

und warum nicht auf node 22?

pages/pwa-results.vue Outdated Show resolved Hide resolved
package.json Show resolved Hide resolved
components/ChatVisualization/Chat.vue Show resolved Hide resolved
components/ChatVisualization/PdfDownloadPopup.vue Outdated Show resolved Hide resolved
Copy link
Member

@fellnerse fellnerse left a comment

Choose a reason for hiding this comment

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

fertig

@fellnerse fellnerse merged commit 6751c1d into dev May 31, 2024
4 checks passed
@fellnerse fellnerse deleted the fixes branch May 31, 2024 07:39
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