[SevenTV/ServerGo] branch deleted: dependabot/github\_actions/dev/docker/build\-push\-action\-2\.5\.0
#github
1 messages ยท Page 54 of 1
e65d55b API/GQL (User) - Twitch Broadcast&FollowerCount - AnatoleAM
[SevenTV/ServerGo] New branch created: feat/gql\-twitch\-stream\-data
Adding two new fields to the User GQL type
- Broadcast: the user's active live broadcast, if any
- Follower Count: the user's current amount of followers
This is used for the User Page on the Web App.
And do we need all the fields in the broadcaster
For what purpose?
Displaying more contextual information on 7tv.app user pages
And do we need all the fields in the broadcaster
Some of them likely won't be used but since it's public information provided by twitch there's no harm in allowing them to be queried
The initial response from twitch is a list of streams, so it's streams.
eae2a68 GQL - Add Broadcast & FollowerCount Query to Us... - AnatoleAM
39c4e93 Change stage deploy branch - AnatoleAM
[SevenTV/App] New branch created: fix/change\-stage\-deploy\-branch
Changing the branch to deploy to stage from dev to stage
f3caf31 Fix PR branch workflow - AnatoleAM
8d8f850 Change POST -> PATCH - AnatoleAM
5f83bd4 Use param name insstead of body - AnatoleAM
cc93cc5 Remove push branch list - AnatoleAM
8b128b3 Remove push trigger - AnatoleAM
7041359 Use types synchronize, edited - AnatoleAM
5719314 Use 'allow any branch' label to skip - AnatoleAM
c722f08 Moved image uploader confirmation setting to ex... - Mm2PL
991892e Add /openurl command (#2926) - talneoran
07231e5 Fix large timeout duration in moderation button... - talneoran
e1895de Hide completion popup when cursor is at first p... - talneoran
db09a88 Add clickable usernames in host offline message... - talneoran
chatterino guy 
[SevenTV/chatterino7] tag deleted: nightly\-build
[SevenTV/chatterino7] New tag created: nightly\-build
[SevenTV/App] New branch created: feat/user\-page\-stream\-data
d389704 Show live status on user page - AnatoleAM
Adding indicators about a user's live status
3027c1f Bump github.com/spf13/viper from 1.8.0 to 1.8.1 - dependabot[bot]
6a0c123 Bump github.com/aws/aws-sdk-go from 1.38.64 to ... - dependabot[bot]
426fd60 Merge pull request #38 from SevenTV/dependabot/... - TroyDota
d9dfd0f Merge pull request #39 from SevenTV/dependabot/... - TroyDota
eae2a68 GQL - Add Broadcast & FollowerCount Query to Us... - AnatoleAM
[SevenTV/App] branch deleted: feat/user\-page\-stream\-data
[SevenTV/App] branch deleted: fix/change\-stage\-deploy\-branch
All changes look good
All changes look good
243592e fix(deps): bump @nguniversal/express-engine fro... - dependabot[bot]
[SevenTV/App] New branch created: dependabot/npm\_and\_yarn/nguniversal/express\-engine\-12\.0\.2
Bumps @nguniversal/express-engine from 12.0.1 to 12.0.2.
Commits
659cf0e release: v12.0.2
ab6906c fix(@โnguniversal/express-engine): remove in-existing migrations reference
4856732 docs: update command to generate changelog
17f58bc build(deps): bump dns-packet from 1.3.1 to 1.3.4
See full diff in compare view
[ it provides updated contextual data about the mutated object.
Furthermore, there are some cases where client's input isn't in line with how the server changes the data. For example, a numeric value of 0 might reset a value to its default, and the server would return that default value rather than 0. This wo...
Not cached since the cached data might be outdated if multiple mutations occur before cache's expire
WHy doesnt it have tags?
Why is it there DinkDonk
Check new commits FeelsDankMan
No, i mean there is a GQL mutation.
Its fine but returning a full user is just weird i think
Its fine but returning a full user is just weird i think
It lets you select what you want to return with what you modified
[SevenTV/ServerGo] branch deleted: feat/gql\-mutate\-user
3580ad8 ModQ - Prevent images from overflowing in height - AnatoleAM
[SevenTV/App] New branch created: feat/admin\-area\-update
769c742 Split permission for setting global emotes - AnatoleAM
5ae1c0e Merge branch 'dev' of github.com:SevenTV/Server... - AnatoleAM
d46b7cb datastructure - Notification type definition - AnatoleAM
Implementing a complete notification system, creating a direct line of communication to end-users for alerts such as moderation actions or other messages relevant to a user or group of users.
a766713 API/GQL - User Mutation (#41) - AnatoleAM
769c742 Split permission for setting global emotes - AnatoleAM
5ae1c0e Merge branch 'dev' of github.com:SevenTV/Server... - AnatoleAM
07648d8 API/GQL - Add Roles to meta query - AnatoleAM
[SevenTV/ServerGo] New branch created: feat/gql\-meta\-roles
[SevenTV/App] New branch created: feat/user\-editing
Adding the full list of available application-wide roles to the GQL meta query
Adding UI components to allow editing users
d6c973e API/GQL - Check actor is above target new role ... - AnatoleAM
[SevenTV/ServerGo] branch deleted: feat/gql\-meta\-roles
[SevenTV/App] branch deleted: feat/user\-editing
f995d63 API/GQL - Add Roles to meta query (#43) - AnatoleAM
@AnatoleAM i was asleep BrokeBack
@AnatoleAM i was asleep BrokeBack
I know I just pushed it live since eggy wanted her role perms LULE, LMK if something needs change tho
499cb88 API/GQL (Search Emotes) - Width Range Filter - AnatoleAM
[SevenTV/ServerGo] New branch created: feat/emote\-search\-by\-size
Adding a new filter parameter to find emotes by minimum and maximum width
Chatterino 7.3.3 (commit 20a71273e)
When I drag chatterino to the edge of the screen on windows 10 to make it half the screen, the left edge is partly on my other monitor and the bottom is partly off screen.
I hope this video makes sense
This occurs on the main Chatterino app as well. Might want to open this issue there instead.
This is a upstream issue, nothing to do with the chatterino7 fork
fc15ee2 API/REST - webext/platforms endpoint variants - AnatoleAM
[SevenTV/App] New branch created: feat/third\-party\-platforms
c94caef API/GQL (Search Emotes) - Width Range Filter (#44) - AnatoleAM
[SevenTV/ServerGo] branch deleted: feat/emote\-search\-by\-size
cb3173c Home - Mobile Apps & Third-Party Tools - AnatoleAM
c9ade67 API/REST - /platforms/:platform/:variant? route - AnatoleAM
Enabling the "Mobile" and "More" icons and building UI components to display these item
[SevenTV/App] branch deleted: feat/third\-party\-platforms
0fd0b8c Home - Mobile Apps & Third-Party Tools (#40) - AnatoleAM
77bac74 fix(deps): bump docker/build-push-action from 2... - dependabot[bot]
[SevenTV/App] New branch created: dependabot/github\_actions/docker/build\-push\-action\-2\.6\.1
Bumps docker/build-push-action from 2.5.0 to 2.6.1.
Release notes
Sourced from docker/build-push-action's releases.
v2.6.1
Small typo and ensure trimmed output (#400)
v2.6.0
Handle git sha version of buildx (#399)
Update dev deps (#393)
Use built-in getExecOutput (#392)
Use core.getBooleanInput (#391)
Bump csv-parse from 4.15.4 to 4.16.0 (#385)
Bump @โactions/exec from 1.0.4 to 1.1.0 (#382)
Bump @โactions/core from 1.3.0 to 1.4.0 (#383)
Bump...
39d396f chore(deps-dev): bump @types/mongodb from 3.6.8... - dependabot[bot]
[SevenTV/App] New branch created: dependabot/npm\_and\_yarn/types/mongodb\-3\.6\.19
2df173d chore(deps-dev): bump jasmine-core from 3.6.0 t... - dependabot[bot]
[SevenTV/App] New branch created: dependabot/npm\_and\_yarn/jasmine\-core\-3\.8\.0
Bumps jasmine-core from 3.6.0 to 3.8.0.
Release notes
Sourced from jasmine-core's releases.
3.8.0
Please see the release notes.
3.7.1
Please see the release notes.
3.7.0
Please see the release notes
Commits
00586e5 Bump version to 3.8.0
27c650e Updated package description
56daa6f Merge branch 'select-spec' of https://github.com/jlpstolwijk/jasmine into main
e4c7d8a new attempt for fixing #1906: add window.location.pathname to hrefs + comment...
ee88...
[SevenTV/App] branch deleted: dependabot/npm\_and\_yarn/jasmine\-core\-3\.7\.1
617ffd4 chore(deps-dev): bump @angular/cli from 12.0.3 ... - dependabot[bot]
[SevenTV/App] New branch created: dependabot/npm\_and\_yarn/angular/cli\-12\.1\.1
Bumps @angular/cli from 12.0.3 to 12.1.1.
Release notes
Sourced from @โangular/cli's releases.
v12.1.1
Commits
... (truncated)
Commits
32986fc release: v12.1.1
17c3dc6 build: fix failing build due to missing import
682d0ec Revert "release: v12.1.1"
446756c release: v12.1.1
2039d62 fix(@โangular-devkit/build-angular): configure webpack target in common config...
dd60228 fix(@โangular-devkit/build-angular): downlevel for await...
shut the fuck up dependabot
3983255 chore(deps-dev): bump @angular-devkit/build-ang... - dependabot[bot]
[SevenTV/App] New branch created: dependabot/npm\_and\_yarn/angular\-devkit/build\-angular\-12\.1\.1
Bumps @angular-devkit/build-angular from 12.0.3 to 12.1.1.
Release notes
Sourced from @โangular-devkit/build-angular's releases.
v12.1.1
Commits
... (truncated)
Commits
32986fc release: v12.1.1
17c3dc6 build: fix failing build due to missing import
682d0ec Revert "release: v12.1.1"
446756c release: v12.1.1
2039d62 fix(@โangular-devkit/build-angular): configure webpack target in common config...
dd60228 fix(@โangular-devkit/b...
[SevenTV/App] branch deleted: dependabot/npm\_and\_yarn/angular\-devkit/build\-angular\-12\.1\.0
No way this is a good idea. This is extremely bad since if 100k users read this this array will be massive and slow mongo down significantly.
You can just compare 2 objectids via equal.
Can this not be done with an aggregated mongo function?
If you wish to comment functions then you should start with the full name of the function in the beginning of the comment. As per the standard of go comments.
ReadBy could be ignored if the notification is set as an announcement
Its far easier to have 2 collections and query via aggregated function.
Also a new user will by default receive all notifications. And if let's say they acquire a role that will get notified of events that were specific to that role in the past.
Best approach I think it's too have a notification. And then to have a table of notification recipients and make one for each user who will get the notification. And once they read it you update the state of that object making it so they don't get pinged again.
And then also don't create notifications for all users at the same time. Do a Cron job.
First of all, image uploader has nothing to do with our fork of Chatterino so if anything, you should've opened an issue on the upstream repository.
Secondly, this doesn't seem like an issue with image uploader, but your config. Did you follow the example configuration on Chatterino wiki?
don't working while link preview is enabled
Could you elaborate on this part? What does link preview has to do with that even?
some ppl
d7ca1ef fix error after deleting the stale yaml - ReneWerner87
9b3662e ๐ฅ Customize the source of session_id (#1159) - leozhantw
bad3abb fix comments in proxy middleware (#1355) - aschenmaker
951d437 Update fasthttp version to resolve brotli warni... - milosgajdos
b14224e Prepare release 2.11.0 - ReneWerner87
Please provide enough information so that others can review your pull request:
Explain the details for making this change. What existing problem does the pull request solve?
Commit formatting
Use emojis on commit messages so it provides an easy way of identifying the purpose or intention of a commit. Check out the emoji cheatsheet here: https://gitmoji.carloscuesta.me/
dank
d7ca1ef fix error after deleting the stale yaml - ReneWerner87
9b3662e ๐ฅ Customize the source of session_id (#1159) - leozhantw
bad3abb fix comments in proxy middleware (#1355) - aschenmaker
951d437 Update fasthttp version to resolve brotli warni... - milosgajdos
b14224e Prepare release 2.11.0 - ReneWerner87
a766713 API/GQL - User Mutation (#41) - AnatoleAM
769c742 Split permission for setting global emotes - AnatoleAM
5ae1c0e Merge branch 'dev' of github.com:SevenTV/Server... - AnatoleAM
f995d63 API/GQL - Add Roles to meta query (#43) - AnatoleAM
fc15ee2 API/REST - webext/platforms endpoint variants - AnatoleAM
c7dc453 document method names for NotificationBuilder - AnatoleAM
An aggregation pipeline could be explored, although those can be time consuming to get right. Something to consider for later?
Is there a reason we need to push this out immediately and not get it right the first time?
What's wrong with this approach exactly? It's still a single query to the DB, just that the program does the work instead of Mongo.
Yes, mongo doing the work is better here i think.
I'm not sure, mongo pipelines are nice for a bunch of things but cross-collection aggregations are really difficult to get right. I have doubts it's worth spending 4-6 hours on it at this time
Step 1: Go to the profile of someone who has a colored name (such as a Twitch Partner or Moderator)
Step 2: Make sure this user has channel editors who have a different user color (For example, a normal user as an editor to a Twitch Partner).
Step 3: Click the editor's username on the left side bar.
Step 4: Notice how the name color does not update to reflect the new user's role.
I have attached a video to demonstrate what I am describing.
Sorry, only users with push access can use that command.
I plan to standardize actions on objects to avoid duplicating code (since everything happens in the context of GQL at the moment), that wasn't meant to be pushed now tho FeelsDankMan
It was moved from restutil.go so that it can be utilized in other places if needed, ie to format messages containing human readable visibility instead of bitmask
It's for the goroutine inside the for loop, it waits for the iterator to complete then queries every emote/user, so this lock is here to make sure the data has all been received before moving onto the resolution
No, you lock twice, once at the top and again this is supposed to be an unlock?
b4b7a15 mongo - add indexes to notifications_read col - AnatoleAM
I've added indexes for target and notification under the "notifications_read" collection, but I believe no index is needed for the notification object, as this collection is only ever read by ObjectID
The upper Lock() call triggers the lock, while the second is there to await the completion of all goroutines inside the loop
FeelsDankMan, no. YOU LOCK THE OUTER LOCK TWICE THIS WILL CAUSE A DEADLOCK
https://github.com/SevenTV/ServerGo/blob/b4b7a15dc572fc3f45098e715c2d587da6ef6091/src/server/api/v2/gql/resolvers/query/user.go#L620
https://github.com/SevenTV/ServerGo/blob/b4b7a15dc572fc3f45098e715c2d587da6ef6091/src/server/api/v2/gql/resolvers/query/user.go#L704
But it doesn't work if you don't lock twice? Because the initial lock call doesn't stop code execution. If i do this the resolvers return null
the locking thing needs to change. All of it.
This extension doesn't work with the Alternative Player for Twitch extension. The same chat window loads in that extension, but 7TV doesn't appear there and none of the emotes from it work. BBTV emotes and features are working though. Is it possible to have a working integration or should this be raised with the Alternative Player for Twitch developer?
[SevenTV/App] New branch created: feat/implement\-merge
Adding UI components for the Emote Merging functionality
[SevenTV/App] branch deleted: feat/implement\-merge
b7d433a DataStructure - AuditLog.Entry.Type EMOTE_MERGE - AnatoleAM
061b1e4 CI - Build PR - AnatoleAM
3f1bba1 Update typings commit - AnatoleAM
a32d548 Update Typings - AnatoleAM
[SevenTV/App] New branch created: feat/notifications
[SevenTV/App] New branch created: feat/update\-angular
[SevenTV/App] branch deleted: feat/notifications
All changes look good
[SevenTV/App] branch deleted: feat/reference\-typings\-as\-submodule
[SevenTV/App] branch deleted: feat/update\-angular
This doesnt have to be a pointer.
I started standardizing common actions on objects so that the code can be re-utilized outside of the GQL environment
This is actually your code, I just copy-pasted - this might be something you consider as legacy? May need a full checkup of the codebase to fix other issues like that
Improves readability
Yeah, probs an issue i made. We can leave it for now and go through again.
It looks Weird. WeirdChamp
Bruh it looks like straight litty fire fax on a stack wdym
No, of why someone would want to merge an emote.
I mean
This is what this whole pull request is about LULW
it allows mods to merge dupes and whatnot, into the original or a higher quality version
I don't think so, since I need the users' IDs. It's used to target notifications
7631315 Handle error at Emotes.Delete() - AnatoleAM
Fixed by 7631315c7c91a753eb921072b9b311eede96ebd9
Fixed in 6f4bd8404b303f9d4740d993498252a5c67ef9b9?
There are 3 notifications being sent to different users / groups of users here
First one is to the owner of the emote being merged: telling them their emote got assrammed and it's gone
Second informs users who had the merged emote enabled that it's been merged into another. (This is fairly unnecessary and can likely be removed in the future, but I figured it'd make for a good stress test)
Third informs the owner of the emote that got merged into that their emote just got X new channels a...
Not sure what you meant with remove the else checks, unless you meant I should unscope the err checks? (that uses more lines)
Yeah but then in theory 1 person gets 3 notifications :WAYTOODANK: from 1 action?
DinkDonk yes, problem? :tf:
[SevenTV/ServerGo] New branch created: feat/linter\-fix
[SevenTV/ServerGo] branch deleted: feat/merge\-emote
The linter fails successfully @AnatoleAM OMEGALUL
From failing positively to succeeding negatively LETS GOOOOOOOOO
For some unknown reason the linter fails silently and doesnt lint.
168f4a8 Bump docker/build-push-action from 1 to 2.6.1 - dependabot[bot]
[SevenTV/ServerGo] New branch created: dependabot/github\_actions/dev/docker/build\-push\-action\-2\.6\.1
Bumps docker/build-push-action from 1 to 2.6.1.
Release notes
Sourced from docker/build-push-action's releases.
v2.6.1
Small typo and ensure trimmed output (#400)
v2.6.0
Handle git sha version of buildx (#399)
Update dev deps (#393)
Use built-in getExecOutput (#392)
Use core.getBooleanInput (#391)
Bump csv-parse from 4.15.4 to 4.16.0 (#385)
Bump @โactions/exec from 1.0.4 to 1.1.0 (#382)
Bump @โactions/core from 1.3.0 to 1.4.0 (#383)
Bump ws ...
585ff89 Bump github.com/aws/aws-sdk-go from 1.38.68 to ... - dependabot[bot]
[SevenTV/ServerGo] New branch created: dependabot/go\_modules/dev/github\.com/aws/aws\-sdk\-go\-1\.39\.0
Bumps github.com/aws/aws-sdk-go from 1.38.68 to 1.39.0.
Release notes
Sourced from github.com/aws/aws-sdk-go's releases.
Release v1.39.0 (2021-07-02)
Service Client Updates
service/ec2: Updates service API
This release removes network-insights-boundary
service/elasticloadbalancingv2: Updates service documentation
SDK Features
internal/ini: The ini parser has been updated to support [, ], :, and = being present in section key values. (#3958)
Releas...
9b7fbc8 Bump go.mongodb.org/mongo-driver from 1.5.3 to ... - dependabot[bot]
[SevenTV/ServerGo] New branch created: dependabot/go\_modules/dev/go\.mongodb\.org/mongo\-driver\-1\.5\.4
Bumps go.mongodb.org/mongo-driver from 1.5.3 to 1.5.4.
Commits
89691b7 Update version to v1.5.4
19a4ed1 GODRIVER-2037 Don't clear the connection pool on client-side connect timeout ...
be28a1c GODRIVER-2057 Fix uri merging on options.ClientOptions (#689)
e0ed5a0 Update version to v1.5.4+prerelease
See full diff in compare view
[
Add tls config for http client in proxy middleware (#1393)
Added decompression cases for Body function (#1402)
๐งน Updates
Remove duplicate line in test (#1404)
Deprecate feature-policy (#1392)
Improved some conditions (#1386)
๐ Fixes
Session middleware issues (#1407)
๐ Documentat...
86a1ce0 Bump github.com/go-redis/redis/v8 from 8.10.0 t... - dependabot[bot]
[SevenTV/ServerGo] New branch created: dependabot/go\_modules/dev/github\.com/go\-redis/redis/v8\-8\.11\.0
Bumps github.com/go-redis/redis/v8 from 8.10.0 to 8.11.0.
Release notes
Sourced from github.com/go-redis/redis/v8's releases.
v8.11.0
Change
Remove OpenTelemetry metrics, Linked #1534 #1805
New Command
XAutoClaim
ZRangeStore
ZUnion
Command More Options
XAdd: NoMkStream+MinID+Limit
XTrim: MinID+Limit
XGroup: CreateConsumer
ZAdd: GT+LT
ZRange: ByScore+ByLex+Rev+Limit
New API
XAutoClaim(ctx context.Context, a *XAutoClaimArgs) *XAutoClaimCmd
XAutoClaimJ...
[SevenTV/ServerGo] New branch created: dependabot/go\_modules/dev/github\.com/gofiber/websocket/v2\-2\.0\.7
3e36195 Bump github.com/gofiber/websocket/v2 from 2.0.6... - dependabot[bot]
Bumps github.com/gofiber/websocket/v2 from 2.0.6 to 2.0.7.
Release notes
Sourced from github.com/gofiber/websocket/v2's releases.
v2.0.7
๐งน Updates
Bump github.com/gofiber/fiber/v2 from 2.13.0 to 2.14.0 (#55)
Commits
515f409 Merge pull request #55 from gofiber/dependabot/go_modules/github.com/gofiber/...
655aacc Bump github.com/gofiber/fiber/v2 from 2.13.0 to 2.14.0
See full diff in compare view
[
Pull request checklist:
- [X]
CHANGELOG.mdwas updated, if applicable
Description
#19 has a copy and paste error in the numbering of badge slots.
imho no changelog entry is needed as no functionality is changed. Just a very small clean up. :)
chatterino compiling
4ea1a30 corrected numbering in badge slot comments (#26) - TranRed
70f1a2c Bump github.com/gofiber/websocket/v2 from 2.0.6... - dependabot[bot]
[SevenTV/ServerGo] branch deleted: dependabot/go\_modules/dev/github\.com/gofiber/websocket/v2\-2\.0\.7
c722f08 Moved image uploader confirmation setting to ex... - Mm2PL
991892e Add /openurl command (#2926) - talneoran
07231e5 Fix large timeout duration in moderation button... - talneoran
e1895de Hide completion popup when cursor is at first p... - talneoran
db09a88 Add clickable usernames in host offline message... - talneoran
[SevenTV/chatterino7] tag deleted: nightly\-build
[SevenTV/chatterino7] New tag created: nightly\-build
a45528e mongo - Fix notifications collection create call - AnatoleAM
7f1c125 mongo - ignore Err for create notify collection - AnatoleAM
a8a0742 Only restrict Highlight color for whispers in m... - Mm2PL
0dd545f fix: provide better UX for HTML color picker ed... - leon-richardt
8385d06 Fix logging category names (#2948) - Mm2PL
6830e8b Readd removing of unmodified cache files (#2947) - talneoran
804bc52 fix overflows in the max image size code (#2950) - fourtf
4ea1a30 corrected numbering in badge slot comments (#26) - TranRed
[SevenTV/chatterino7] tag deleted: nightly\-build
[SevenTV/chatterino7] New tag created: nightly\-build

[SevenTV/ServerGo] New branch created: feat/restrict\-gql\-notify\-permissions
Adding permission checks for gql notification resolvers, preventing any user from reading another users' notifications unauthorized.
9220fb3 tag bson omitempty for Notification.Read - AnatoleAM
55626fd User notification count resolver - AnatoleAM
Wait on that last commit i donked it
c117f3e User notification count resolver - AnatoleAM
Read isn't a persistent data point of the notification structure, it's always inferred from read states.
why is this commit here? was this not for the previous pr?
Can we not do a query here rather than fetching them.
This came to mind yeah, do you think it'd be more efficient to query two times (if both count and notification objects are requested) rather than fetch every matching notification?
bd01a0d Split queries for notification count/data - AnatoleAM
I changed it to a cached collection count query for notification_count. This should be good for deploy now
66e6f0a chore(deps-dev): bump @types/jasmine from 3.6.4... - dependabot[bot]
[SevenTV/App] New branch created: dependabot/npm\_and\_yarn/types/jasmine\-3\.7\.8
[SevenTV/App] branch deleted: dependabot/npm\_and\_yarn/types/jasmine\-3\.7\.7
1480692 chore(deps-dev): bump @types/mongodb from 3.6.8... - dependabot[bot]
[SevenTV/App] New branch created: dependabot/npm\_and\_yarn/types/mongodb\-3\.6\.20

