Skip to content

Allow multiple webhooks and Instagram accounts #46

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

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

RyanLua
Copy link
Owner

@RyanLua RyanLua commented Feb 5, 2025

Fixes #20

Add support for multiple webhooks and Instagram accounts.

  • Parser Updates:

    • Update parser.add_argument for instagram_username to accept multiple usernames.
    • Update parser.add_argument for discord_webhook_url to accept multiple URLs.
    • Update help text to reflect the new functionality.
  • Main Logic Updates:

    • Update args.instagram_username to handle a list of usernames.
    • Update args.discord_webhook_url to handle a list of URLs.
    • Update check_for_new_posts function to iterate over multiple Instagram accounts and webhooks.
    • Update send_to_discord function to handle multiple webhooks.
  • Documentation Updates:

    • Update docs/source/usage.rst to show usage with multiple Instagram usernames and Discord webhook URLs.
    • Update docs/source/getting-started.rst to reflect the new functionality of specifying multiple webhooks or Instagram accounts.

Checklist

  • Improve logging message to be more readable:
    02/05/2025 10:16:04 PM - INFO - Monitoring '['raenlua']' every 3600 seconds on ̀['https://discord.com/api/webhooks/1336821555593871451/pXDkxuqdOJ3VLi8jUFbiXco91_GFKPKC_emRNF3KvVaU3wJkuqWhgB25yollvLoHaR-D']
  • Test for edge cases

For more details, open the Copilot Workspace session.

Fixes #20

Add support for multiple webhooks and Instagram accounts.

* **Parser Updates:**
  - Update `parser.add_argument` for `instagram_username` to accept multiple usernames.
  - Update `parser.add_argument` for `discord_webhook_url` to accept multiple URLs.
  - Update help text to reflect the new functionality.

* **Main Logic Updates:**
  - Update `args.instagram_username` to handle a list of usernames.
  - Update `args.discord_webhook_url` to handle a list of URLs.
  - Update `check_for_new_posts` function to iterate over multiple Instagram accounts and webhooks.
  - Update `send_to_discord` function to handle multiple webhooks.

* **Documentation Updates:**
  - Update `docs/source/usage.rst` to show usage with multiple Instagram usernames and Discord webhook URLs.
  - Update `docs/source/getting-started.rst` to reflect the new functionality of specifying multiple webhooks or Instagram accounts.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/RyanLua/InstaWebhooks/issues/20?shareId=XXXX-XXXX-XXXX-XXXX).
@Copilot Copilot AI review requested due to automatic review settings February 5, 2025 21:54
Copy link

github-actions bot commented Feb 5, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

Files not reviewed (2)
  • docs/source/getting-started.rst: Language not supported
  • docs/source/usage.rst: Language not supported

@RyanLua RyanLua self-assigned this Feb 5, 2025
@RyanLua RyanLua added the bug Something isn't working label Feb 5, 2025
@RyanLua RyanLua added this to the v0.1.4 milestone Feb 5, 2025
@RyanLua RyanLua marked this pull request as draft February 5, 2025 22:31
@RyanLua RyanLua added enhancement New feature or request and removed bug Something isn't working labels Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow multiple webhooks and Instagram accounts
1 participant