Skip to content

style(frontend): make notification cards fully clickable on landing page - #7750

Merged
mergify[bot] merged 2 commits into
modrinth:mainfrom
Fryrvo:notifpage
Nov 3, 2026
Merged

mergify[bot] merged 2 commits into
modrinth:mainfrom
Fryrvo:notifpage

Conversation

@Fryrvo

@Fryrvo Fryrvo commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

before

Peek 2026-09-29 13-00

after

Peek 2026-09-29 12-59

@github-actions

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@Hallskii
Hallskii requested a review from IMB11 October 2, 2026 16:47
@Ceebug

Ceebug commented Oct 7, 2026

Copy link
Copy Markdown

Looks good

@Prospector

Copy link
Copy Markdown
Member

@Mergifyio queue

@mergify

mergify Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

🛑 Queue command cancelled

@Fryrvo

Fryrvo commented Oct 18, 2026

Copy link
Copy Markdown
Contributor Author

can someone re run CI please?

@Ceebug

Ceebug commented Oct 18, 2026 via email

Copy link
Copy Markdown

@Fryrvo

Fryrvo commented Oct 18, 2026

Copy link
Copy Markdown
Contributor Author

Why is ci not auto run?
…
________________________________ From: Cath @.> Sent: Sunday, 18 October 2026 21:02:49 To: modrinth/code @.> Cc: Ceebug @.>; Comment @.> Subject: Re: [modrinth/code] style(frontend): make notification cards fully clickable on landing page (PR #7750) [https://avatars.githubusercontent.com/u/186661350?s=20&v=4]Fryrvo left a comment (modrinth/code#7750)<#7750 (comment)> can someone re run CI please? — Reply to this email directly, view it on GitHub<#7750?email_source=notifications&email_token=BJQB4SA4XJ42PW374OVB2LD5USIMTA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTMMRUHA2TGNRWGE22M4TFMFZW63VHMNXW23LFNZ2KKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-6248536615>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJQB4SDA6IIMY6KK7SRYDRD5USIMTAVCNFSNUABFKJSXA33TNF2G64TZHMZTOOBXGQ2DKOJUHNEXG43VMU5TKNRSGY2TMOJUGY42C5QC. You are receiving this because you commented.Message ID: @.***>

it failed because it ranout of memory =(

@Ceebug

Ceebug commented Oct 18, 2026 via email

Copy link
Copy Markdown

@mergify

mergify Bot commented Oct 20, 2026

Copy link
Copy Markdown

queue

🛑 The queue request has been cancelled: new commits were pushed

Details

@Fryrvo pushed to this pull request after queue was requested. Review the new commits and request it again.

@Fryrvo
Fryrvo requested a review from Prospector October 20, 2026 09:02
@Prospector

Copy link
Copy Markdown
Member

@Mergifyio queue

@mergify

mergify Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

✅ Merged into main as becf6e8 at 04:24 UTC

8 min in the queue, 8 min of it CI.

So far

2 merge conditions met
  • github-review-approved · required by GitHub branch protection
  • any of · required by GitHub branch protection:
    • check-success = Lint and Test
    • check-neutral = Lint and Test
    • check-skipped = Lint and Test

Times in UTC · dashboard · how the merge queue works

@mergify mergify Bot added the queued label Nov 3, 2026
@mergify
mergify Bot merged commit becf6e8 into modrinth:main Nov 3, 2026
11 checks passed
@mergify mergify Bot removed the queued label Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants