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

Dev #67

Merged
merged 14 commits into from
Feb 1, 2025
Merged

Dev #67

merged 14 commits into from
Feb 1, 2025

Conversation

thefourcraft
Copy link
Member

This pull request includes several changes to the workflow configuration files and Python scripts to improve file path handling and add debugging information.

Workflow Configuration Updates:

  • .github/workflows/post.yml: Updated the dependency installation command to use python -m pip instead of pip directly.
  • .github/workflows/sync-idf-telegram.yml and .github/workflows/sync-tzevaadomm-telegram.yml: Added shell: bash to the session file check step and updated the session file copy command to use $GITHUB_WORKSPACE for the destination path. [1] [2]

Python Script Updates:

  • code/export_posts.py: Changed the GITHUB_REPO_PATH to use a relative path pointing to the parent directory.
  • code/telegram/export_idf_telegram.py: Improved path handling by using os.path.join for SAVE_PATH and STATE_FILE, and updated the Telegram client initialization to use an absolute path for the session file. Added print statements for debugging the state file operations. [1] [2] [3]

@thefourcraft thefourcraft merged commit f2b20be into production Feb 1, 2025
1 of 2 checks passed
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.

1 participant