Repository navigation
Create sync model for webhook - #1736
Merged
Merged
Conversation
* This is used for Authors to track the internal processing status of each script. * Only shows up per Author i.e. nobody else and you *must* be logged in to view * Search might be incorrect atm... still tinkering with that part. * Hide some pre-existing GH items if no auth strategy present. * Eventually more is needed but this is a start. Applies to OpenUserJS#1730 and a few related others in the past; Will eventually affect OpenUserJS#430 by moving debugging to Author page instead of OUJS intervention.
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Apr 15, 2021
* Reorder some panels * New minimized greyscale asset favicon svg for usage as "text" prefixes. Not going to bother with individual sizes. Applies to OpenUserJS#1705 and post OpenUserJS#1736
Martii
added a commit
that referenced
this pull request
Apr 15, 2021
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Apr 15, 2021
* Use *bootstrap* accordion to hide the instructions until clicked/tapped. If/when the feature gets reenabled they might be longer for GH. * Symmetry * Some bolding elsewhere * Verbage on previously published... webhook currently sends all push notices whether published or not. Applies to OpenUserJS#1705 and post OpenUserJS#1736
Merged
Martii
added a commit
that referenced
this pull request
Apr 15, 2021
* Use *bootstrap* accordion to hide the instructions until clicked/tapped. If/when the feature gets reenabled they might be longer for GH. * Symmetry * Some bolding elsewhere * Verbage on previously published... webhook currently sends all push notices whether published or not. Applies to #1705 and post #1736 Auto-merge
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Apr 15, 2021
* No reason to have two links on same page. Larger chance at being out of sync at some point. * Tweak instructions and formatting. Applies to OpenUserJS#1705 and post OpenUserJS#1736
Martii
added a commit
that referenced
this pull request
Apr 15, 2021
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Apr 16, 2021
* Also add in additional check missed in OpenUserJS#1736 with forms Applies to OpenUserJS#1705
Merged
Martii
added a commit
that referenced
this pull request
Apr 16, 2021
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Jan 17, 2022
* Scoot syncs closer to scripts since they are related. Post OpenUserJS#1736 * Expose previous counts to the authed user... still tracking at admin+ but may allow mod+ for just the counts but not content for mods... still pondering... votes may not be necessary and they already have access to "anonymous" flagging... thinking out loud... still pondering. * Flags include historical count regardless of action taken. This will eventually change. Post OpenUserJS#1942 OpenUserJS#785 NOTE: * Btw AuthAs is great however it's really annoying at times *(think reauthentication confirmations)* so this is why Admin+ have quicker access to see what's going on without authAs then end session and restart a new Admin+ session.
Martii
added a commit
that referenced
this pull request
Jan 17, 2022
* Scoot syncs closer to scripts since they are related. Post #1736 * Expose previous counts to the authed user... still tracking at admin+ but may allow mod+ for just the counts but not content for mods... still pondering... votes may not be necessary and they already have access to "anonymous" flagging... thinking out loud... still pondering. * Flags include historical count regardless of action taken. This will eventually change. Post #1942 #785 NOTE: * Btw AuthAs is great however it's really annoying at times *(think reauthentication confirmations)* so this is why Admin+ have quicker access to see what's going on without authAs then end session and restart a new Admin+ session. Auto-merge
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Applies to #1730 and a few related others in the past; Will eventually affect #430 by moving debugging to Author page instead of OUJS intervention.
Misc note:
consolestatements, at least here, but in for posterity in case someones IDE might.