Feature: mint notifications#16
Merged
Merged
Conversation
zacck-zz
force-pushed
the
feature/mint_notifications
branch
from
f859b11 to
f076980
Compare
lucca65
approved these changes
Nov 14, 2019
Member
lucca65
left a comment
lucca65
left a comment
There was a problem hiding this comment.
This looks good, as always. One thing pops up to my mind though:
As you can see here the issue action is an inline action. Our demux solution currently don't support inline actions so we won't have the feature done in the end.
I think the best course of action is to finish up this and the frontend pull request and then start working on the Demux side to see if we can somehow get inline actions. Maybe there are updates to demux that I'm not aware of!
Member
|
In anyway, this PR is approved, you can merge it! |
Contributor
Author
|
@lucca65 we should open an issue to deal with the demux situation ... |
lucca65
pushed a commit
that referenced
this pull request
May 17, 2026
* Minor feature bump for issue/mint notifications * Create function and trigger to run when a mint is added to the database * Build datastructure for mint and add mint_factory for testing * Add test for mint notification and format file to remove trailing spaces * Amend variable and alert information * Add a callback to handle mint triggers * Add a function to handle notifying mint receivers * Add mints to the public schema * Expose mints from a community
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.
Changes Proposed