feat: point the per-PR logs link directly at the activity log#601
Merged
Conversation
Contributor
Author
|
This pull request is part of a Mergify stack:
|
Contributor
Merge Protections🟢 All 7 merge protections satisfied — ready to merge. Show 7 satisfied protections🟢 ⛓️ Depends-On RequirementsRequirement based on the presence of
🟢 🤖 Continuous Integration
🟢 👀 Review Requirements
🟢 Enforce conventional commitMake sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 Reviews
🟢 📕 PR description
🟢 🚦 Auto-queueWhen all merge protections are satisfied, this pull request will be queued automatically. |
mergify
Bot
force-pushed
the
devs/JulianMaurin/julian/mrgfy-7878-browser-extension-batch-row-queue-link-deep-links-to-the/deep-link-logs-link-batch-filtered-activity-log--5c6a7201
branch
from
July 13, 2026 09:34
70f235e to
557287c
Compare
Contributor
|
A predecessor in this stack was just merged, but I couldn't auto-rebase this PR: there's a conflict that needs human resolution. Conflict in: Resolve with |
Contributor
|
@JulianMaurin this pull request is now in conflict 😩 |
The logs link on non-batch PRs targeted the retired /event-logs URL, riding the dashboard's redirect + param-translation shim (pullRequestNumber → pull_request) and landing on the activity log's default 1-hour window — empty for any PR queued earlier. Target /activity-log directly with pull_request and preset=Past1month, through a URL builder shared with the batch link. getEventLogLink() becomes getPullRequestActivityLogLink(), named after its destination. The "logs" label is unchanged. MRGFY-7878 Test plan: - npx jest (279 passing) — non-batch PRs assert the direct activity-log URL with pull_request and preset; batch link unchanged. - npx biome check — clean. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QyefSTmdtUKXCSbYRJvSQu Change-Id: Idcbe837c50b3be3ab05a8e875116804ae977a822
JulianMaurin
force-pushed
the
devs/JulianMaurin/julian/mrgfy-7878-browser-extension-batch-row-queue-link-deep-links-to-the/point-per-pr-logs-link-directly-activity-log--dcbe837c
branch
from
July 14, 2026 10:05
faed6d0 to
15b273b
Compare
JulianMaurin
force-pushed
the
devs/JulianMaurin/julian/mrgfy-7878-browser-extension-batch-row-queue-link-deep-links-to-the/deep-link-logs-link-batch-filtered-activity-log--5c6a7201
branch
from
July 14, 2026 10:05
557287c to
b2c998c
Compare
Contributor
Author
Revision history
|
JulianMaurin
marked this pull request as ready for review
July 15, 2026 14:06
Base automatically changed from
devs/JulianMaurin/julian/mrgfy-7878-browser-extension-batch-row-queue-link-deep-links-to-the/deep-link-logs-link-batch-filtered-activity-log--5c6a7201
to
main
July 16, 2026 09:51
mergify
Bot
force-pushed
the
devs/JulianMaurin/julian/mrgfy-7878-browser-extension-batch-row-queue-link-deep-links-to-the/point-per-pr-logs-link-directly-activity-log--dcbe837c
branch
from
July 16, 2026 09:51
15b273b to
92efe3f
Compare
kozlek
approved these changes
Jul 16, 2026
AlexandreGaubert
approved these changes
Jul 16, 2026
Contributor
Merge Queue Status
This pull request spent 11 seconds in the queue, including 2 seconds running CI. Required conditions to merge
|
mergify
Bot
deleted the
devs/JulianMaurin/julian/mrgfy-7878-browser-extension-batch-row-queue-link-deep-links-to-the/point-per-pr-logs-link-directly-activity-log--dcbe837c
branch
July 16, 2026 12:36
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
3 participants
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.
The logs link on non-batch PRs targeted the retired /event-logs URL, riding
the dashboard's redirect + param-translation shim (pullRequestNumber →
pull_request) and landing on the activity log's default 1-hour window —
empty for any PR queued earlier. Target /activity-log directly with
pull_request and preset=Past1month, through a URL builder shared with the
batch link. getEventLogLink() becomes getPullRequestActivityLogLink(),
named after its destination. The "logs" label is unchanged.
MRGFY-7878
Test plan:
with pull_request and preset; batch link unchanged.
Co-Authored-By: Claude Fable 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01QyefSTmdtUKXCSbYRJvSQu
Depends-On: #600