-
Notifications
You must be signed in to change notification settings - Fork 1
Personal improvements #1
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
Merged
Merged
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
b2215ea
Fix SQL injection vulnerabilities and improve database operations
NiXTheDev cc4102c
Add custom404ImagePath to settings: schema, types, and models
NiXTheDev 0ada39b
Add settings menu and 404 image upload handler
NiXTheDev ba0bd93
Integrate custom 404 image into entry viewing
NiXTheDev 315fe77
Add testing for custom 404 image settings
NiXTheDev 460c523
Fix linting issues and clean up unused imports
NiXTheDev af0e3bc
Fix custom API URL configuration for file downloads and API calls
NiXTheDev 8a928a0
Improve user experience with clearer prompts, examples, and help text
NiXTheDev a16ab51
Update README with configuration instructions
NiXTheDev 48b4a23
Add configurable Telegram API base URL support
NiXTheDev 27b8203
Fix custom API URL configuration for file downloads and API calls
NiXTheDev e81b20e
Fix database creation by ensuring file exists before table creation
NiXTheDev 15eaa4c
Fix HTML formatting by adding parse_mode to all HTML messages
NiXTheDev c39595e
Add fallback to official Telegram API for file downloads when custom …
NiXTheDev 9f0f626
Fix file path handling for Telegram file downloads - extract relative…
NiXTheDev 7b38a54
Fix settings initialization - ensure settings record exists before up…
NiXTheDev f427ee1
Add error resilience to conversations - retry mechanisms, graceful er…
NiXTheDev 0618691
Implement auto-retry functionality with fallback API support and conv…
NiXTheDev 1859672
Adding a way to view journal entries.
CodeCanna b9e3137
Removed a line saying selfie features arent working.
CodeCanna 41e133f
Fixed reply message to say data instead of entries
CodeCanna 2bda40d
Added view_journal_entries conversation.
CodeCanna ba50f91
code cleanup
CodeCanna 850e2f9
added /bin to .gitignore to store deno binaries
CodeCanna d13fa18
Merge upstream changes (dd2d80f-43ebb2e)
NiXTheDev dfde22e
fix: deno fmt
NiXTheDev d3c693c
Fix linting errors: remove unused imports and replace any types with …
NiXTheDev fcc2124
Fix type errors: properly handle nullable database fields and PRAGMA …
NiXTheDev fb2bcd5
fix: deno fmt\nagain...
NiXTheDev b29ca2a
Replace console.log with proper logging framework
NiXTheDev d8aad1b
Consolidate database connections to use withDB helper
NiXTheDev 0f808f8
Refactor: Fix non-null assertions, add validation, remove dead code, …
NiXTheDev dee3628
Refactor: Standardize error handling patterns
NiXTheDev 018c7cb
Refactor: Optimize database integrity checks and error messages
NiXTheDev 53591ba
Refactor: Improve error messages for better UX
NiXTheDev 414034a
fix: deno lint
NiXTheDev 34f4524
fix: @std/log version pinned to ^0.224.14 instead of ^0.224
NiXTheDev 60411f9
fix: deno fmt
NiXTheDev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1,2 @@ | ||
| export const pdfFontSize = 30; | ||
| export const MAX_FILE_SIZE_BYTES = 10 * 1024 * 1024; // 10MB limit for file uploads |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| SELECT seq FROM sqlite_sequence WHERE name='<TABLE_NAME>'; | ||
| SELECT MAX(id) as max_id FROM <TABLE_NAME>; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you forget to add this to .gitignore? :3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
likely, but then we need to gitignore all images
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I would think we would ignore all images in the custom 404 image directory because those are dynamic user settings, then we can set the default 404.png in /assets to a more neutral image?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I suppose so, though I we need a separate location for the default
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also I'm not sure if I'm doing something wrong, but the bot won't run on my laptop after merging the changes. I can see all of the tests passed in Github so that's why I think it's something I'm doing. I'm getting
When I add that dependency I get
Am I doing something wrong here? To be honest this is the first time someone has contributed to one of my projects like this, so I could be missing something. If all of your tests passed it should run for me, before I started making changes I thought I should ask.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know how that happened, but i am already working on a fix