Refresh media comment action payload#288
Merged
Merged
Conversation
e82a2e3 to
9142344
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
media_comment()write payload frominstagrapi, including feed action context fieldscountquery param is overridden instead of duplicatedcommentlive workflow target and update the live comment test to upload/read back/cleanup its own media fixtureMirrors subzeroid/instagrapi#2538.
Tests
.venv/bin/python -m pytest tests/regression/test_comments.py::CommentRepliesRegressionTestCase::test_media_comment_posts_current_action_context tests/regression/test_live_smoke.py::LiveSmokeRegressionTestCase::test_fetch_accounts_overrides_existing_default_count -q.venv/bin/python -m pytest tests/regression/test_comments.py tests/regression/test_live_smoke.py -q.venv/bin/python -m ruff check aiograpi tests.venv/bin/python -m ruff format --check aiograpi tests.venv/bin/python -m pytest -sv tests/legacy.py::ExtractorsRegressionTestCase tests/legacy.py::ImageUtilSafeRemoteFetchTestCase tests/legacy.py::DirectExtractorRegressionTestCase tests/legacy.py::PublicRegressionTestCase tests/legacy.py::PolarisProfileNormalizationTestCase tests/legacy.py::CaptchaHandlerMixinRegressionTestCase tests/legacy.py::NoteMixinRegressionTestCase tests/legacy.py::LocationMixinRegressionTestCase tests/legacy.py::ChallengeRegressionTestCase tests/legacy.py::AuthAndStoryRegressionTestCase tests/legacy.py::ClientTestCase tests/legacy.py::DownloadRegressionTestCase tests/legacy.py::DirectMixinRegressionTestCase tests/legacy.py::UserMixinRegressionTestCase tests/legacy.py::TimelineRegressionTestCase tests/legacy.py::StoryConfigureRegressionTestCase tests/legacy.py::UploadRegressionTestCase tests/legacy.py::SignUpTestCase tests/legacy.py::NotificationMixinRegressionTestCase tests/legacy.py::ChapiPortedRegressionTestCase tests/regression.venv/bin/python -m mkdocs build --strict./scripts/check-mypy-baseline.sh.venv/bin/python -m buildLive verification
tests/legacy.py::ClientCommentExtendTestCase::test_media_commentdid not reach the comment endpoint because account setup failed on proxy302 Foundduringlauncher/sync/.TEST_ACCOUNTS_URLas an explicit skip instead of falling back to dummy credentials: https://github.com/subzeroid/aiograpi/actions/runs/26125558847instagrapi: https://github.com/subzeroid/instagrapi/actions/runs/26124367512