Skip to content

[Feature]: Add on crash uploaded SourceMod forward - #42

Merged
sapphonie merged 7 commits into
asherkin:masterfrom
caxanga334:pr-feat-upload-forward
Dec 2, 2025
Merged

[Feature]: Add on crash uploaded SourceMod forward#42
sapphonie merged 7 commits into
asherkin:masterfrom
caxanga334:pr-feat-upload-forward

Conversation

@caxanga334

Copy link
Copy Markdown
Contributor

Description

Adds a SourceMod forward for when a crash is uploaded.

Closes #26

commit c7efce2
Author: caxanga334 <10157643+caxanga334@users.noreply.github.com>
Date:   Sat Nov 22 22:35:24 2025 -0300

    Fix Error

commit e8acf9f
Author: caxanga334 <10157643+caxanga334@users.noreply.github.com>
Date:   Sat Nov 22 22:23:57 2025 -0300

    Fix Possible Memory Leak

commit 51e7189
Author: caxanga334 <10157643+caxanga334@users.noreply.github.com>
Date:   Sat Nov 22 22:13:55 2025 -0300

    Add Upload Forward

@sapphonie sapphonie left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks fine enough, please respond to the review comments. I think I'd also like to see an example plugin using the .inc somewhere, other than that this looks great. Thanks for the PR!

Comment thread extension/forwards.cpp Outdated
Comment thread extension/forwards.cpp Outdated
Comment thread scripting/accelerator.inc Outdated
Forward notifies plugins when the extension is done uploading.
Plugins can fetch data from uploaded crashes via natives.
Added example plugin.
@Kenzzer
Kenzzer self-requested a review November 24, 2025 14:29

@Kenzzer Kenzzer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Amazing work! Just a few nitpicks

Comment thread extension/forwards.cpp
Comment thread scripting/accelerator_example.sp Outdated
Comment thread extension/natives.h Outdated
@caxanga334
caxanga334 requested a review from Kenzzer November 24, 2025 14:59
Removed timer example.
Added example of how to handle plugin late loads.
@sapphonie

Copy link
Copy Markdown
Collaborator

Okay yay

@sapphonie
sapphonie merged commit 87704dc into asherkin:master Dec 2, 2025
5 checks passed
@caxanga334
caxanga334 deleted the pr-feat-upload-forward branch December 2, 2025 21:57
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.

Create sourcepawn forward to know when there's been a crash upload

3 participants