FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update xpath_compile.py by Mekaze99 · Pull Request #6767 · InstaPy/InstaPy · GitHub

Update xpath_compile.py - #6767

Open
Mekaze99 wants to merge 1 commit into
InstaPy:masterfrom
Mekaze99:patch-1
Open

Update xpath_compile.py#6767
Mekaze99 wants to merge 1 commit into
InstaPy:masterfrom
Mekaze99:patch-1

Conversation

Mekaze99 commented Oct 11, 2023
edited
Loading

Copy link
Copy Markdown

Updating file xpath_compile.py to deal with problem with accepting cookies during login

Pull Request Template

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Do not include any personal data.

Fixes # (issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

  • Test

Checklist:

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have checked my code and corrected any misspellings
  • I have performed a self-review of my own code
  • My code follows the style guidelines of this project, black -t py34
  • My changes generate no new warnings

Updating file xpath_compile.py to deal with problem with accepting cookies during login

AxelRP left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

abnerrizzi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

ZeyadFarhat11 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

Copy link
Copy Markdown

Why is this still waiting to be merged?

Mekaze99 commented May 7, 2024

Copy link
Copy Markdown
Author

Idk
I still see "unable to merge" and "review is required"

Copy link
Copy Markdown

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants


Back | FazBrowse Home | New Git URL