Modifying WP fixes for Agent 3#17
Open
D3lK1ch1 wants to merge 12 commits into
Open
Conversation
…of stubs, H1 tags and image without alt texts
| # Requires BOTH the tag AND state=='QA' — a ticket carrying only one (e.g. someone | ||
| # manually retagged or restated it outside the normal Agent 3 fix flow) shouldn't | ||
| # surface as a ready-to-check candidate here. | ||
| resp = requests.post(wiql_url, headers=headers, json={"query": f"SELECT [System.Id] FROM WorkItems WHERE [System.Tags] CONTAINS '{AZURE_QA_TAG}' AND [System.State] = '{qa_state}' ORDER BY [System.Id] DESC"}, timeout=10) |
| with get_db() as conn: | ||
| cursor = conn.cursor() | ||
| placeholders = ','.join(['?'] * len(ticket_ids)) | ||
| cursor.execute(f"DELETE FROM devops_tickets WHERE ticket_id IN ({placeholders})", list(ticket_ids)) |
| for the same cases assert_safe_url() does.""" | ||
| parsed, ip = _resolve_and_validate(url) | ||
| with pin_dns(parsed.hostname, str(ip)): | ||
| return requests.get(url, **kwargs) |
| of crawled-content-derived URLs).""" | ||
| parsed, ip = _resolve_and_validate(url) | ||
| with pin_dns(parsed.hostname, str(ip)): | ||
| return requests.post(url, **kwargs) |
| for the same cases assert_safe_url() does.""" | ||
| parsed, ip = _resolve_and_validate(url) | ||
| with pin_dns(parsed.hostname, str(ip)): | ||
| return requests.get(url, **kwargs) |
| except ValueError as e: | ||
| return jsonify({'success': False, 'error': str(e)}), 400 | ||
| token = base64.b64encode(f':{pat}'.encode()).decode() | ||
| headers = {'Authorization': f'Basic {token}', 'Content-Type': 'application/json'} |
| headers = {'Authorization': f'Basic {token}', 'Content-Type': 'application/json'} | ||
| qa_state = os.getenv('AZURE_QA_STATE', 'QA') | ||
| wiql_url = f'https://dev.azure.com/{org}/{url_quote(project)}/_apis/wit/wiql?api-version=7.1' | ||
| resp = requests.post(wiql_url, headers=headers, json={"query": f"SELECT [System.Id] FROM WorkItems WHERE [System.Tags] CONTAINS '{AZURE_QA_TAG}' ORDER BY [System.Id] DESC"}, timeout=10) |
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.
Added Missing H1 tags and Image Without Alt Text into FIX_MAP.
Among issues, despite the RankMath plugin, missing canonical URL has this certain type of problem: canonical value was written but no tag is rendered on the page — looks like a RankMath output/configuration issue, not a write failure
Everything else works either fixed and can be verified manually with link pointing where, or deferred with explanations requiring human input in tickets.