tunisiano187 / WME-send-to-slack

Script to send (Un)lock/Closures/Open/Validations requests to slack, Discord, Telegram, GForm
https://wmests.bowlman.org
GNU General Public License v3.0
4 stars 7 forks source link

Update WME-send-to-slack.user.js #101

Closed gk1220 closed 1 year ago

gk1220 commented 1 year ago

Fixes # Fix to display the vlidation Icon also when segments are selected; Fix to apply the locklevel from PL

Proposed changes in this pull request:

-


This change is Reviewable

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

gk1220 commented 1 year ago

Hi, I recognised that you merged my PR and pushed it to Greasy Fork without increasing the Version, I think that causes problems with the auto update. For further PR should I include a Version upgrade?

Best regards Gerhard (g1220k)

22.07.2023 18:47:49 Tunisiano18 @.***>:

Merged #101[https://github.com/tunisiano187/WME-send-to-slack/pull/101] into master.

— Reply to this email directly, view it on GitHub[https://github.com/tunisiano187/WME-send-to-slack/pull/101#event-9891400822], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ARO3NYAUF4J77VA4BDCPTB3XRP73HANCNFSM6AAAAAA2TYRHNI]. You are receiving this because you were mentioned.[Verfolgungsbild][https://github.com/notifications/beacon/ARO3NYD3BD3RT3H7LGVS7BTXRP73HA5CNFSM6AAAAAA2TYRHNKWGG33NNVSW45C7OR4XAZNWJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XKUY3PNVWWK3TUL5UWJTYAAAAAETMSZR3A.gif]

GyllieGyllie commented 1 year ago

Hey @gk1220,

Sorry for the late reply. It is easier indeed if you already change the version in your PR. This way we can just merge it and the update goes out.

Kind regards Guillian