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

Various fixes for shortcodes and stability #10

Merged
merged 6 commits into from
Apr 18, 2024
Merged

Various fixes for shortcodes and stability #10

merged 6 commits into from
Apr 18, 2024

Conversation

illgitthat
Copy link
Contributor

I made a few changes on my own fork and wanted to contribute back in case this is still being maintained @dzg.

  • Introduces a fix for issue Inbound text messages from shortcodes are not supported #7 (short codes)
  • Improved the extraction of text from email subjects. This includes better handling of short codes and cases where the number is not found in the subject line.
  • Introduced a new imapSearchFolder configuration parameter. This allows the user to specify a different folder to search for messages, which can be useful if Google Voice emails are automatically archived.
  • Added an example method to run the script using forever in service.sh

I can't guarantee code quality here, but I thought it made sense to open a PR at this point as it solved a few issues I was having.

add support for archived Gmail folder
add sample script to run as a service
Update text body replacement with a regular expression
Attempt to fix TypeError by adding .catch on matrixNotify() function
@dzg dzg merged commit d47ecab into dzg:main Apr 18, 2024
@dzg
Copy link
Owner

dzg commented Apr 18, 2024

Thanks! Good stuff!
I don't really use Beeper any more so I won't be very active here.

@dzg
Copy link
Owner

dzg commented Apr 18, 2024

Oh and – if you're up to it – someone changed the auth method to xoauth2 over at
JackHMcD@2adbcd4

... which is probably a much better (more secure) method than app passwords.

Maybe you have the werewithal to integrate that!

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