-
Notifications
You must be signed in to change notification settings - Fork 14
fix get_statements #218
base: master
Are you sure you want to change the base?
fix get_statements #218
Conversation
WalkthroughThe update in the Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on X ? TipsChat with CodeRabbit Bot (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| @property | ||
| def full_statement_op(self): | ||
| return self._soup.find("a").attrs["data-op"] | ||
| return self._soup.find("a", attrs={"href": "javascript:;"}).attrs["data-op"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a test for this fix.
|
This PR fixes the |
I'll fix the tests and merge it. In the meantime, you can try to use |
Summary by CodeRabbit