-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New tag message is shown when a tag is deleted #232
Comments
For posterity: Commands used (and duplicates because duplicate webhooks..):
Content of the pastee:
|
I would like to work on it. |
Awesome, thanks @Quaqmre 👍 |
Hi guys , I was working on this issue then I find some that ,It is not related webhoos or Mattermost api , when u try to delete tag remotely like ' git push --delete anytag' that is deleted gitlab remoot repo but anytag still there in your local repo. And when u try to push or delete any tag git add the *anytag" again.Then u saw 2 message on your channel 1 message about deleted mesage the other one is brendnew tag with the same name deleted one. Because u have the deleted tag in your local repo.When u delete tags remotely u have to delete in your local repo. I will upload screenshot tomorrow morning :) |
@jsmestad In order to delete a tag locally and remotely, you have to also run |
That seems like the expected behavior to me. |
@hanzei How should we proceed about this issue ? Can you help me what should i do ? |
@Quaqmre The example looks good to me. The notifications appear in the right order. Would you please create a WIP PR for an early code review? |
@hanzei Actually I dont know how can I create WIP PR , I try to create PR but I dont have any commit or changes ,so I cant create PR without changes what sould I do ? ` Showing with 0 additions and 0 deletions. No commit comments for this range |
@Quaqmre Are you saying that you are unable to reproduce the issue? Unfortunately, I cannot test this as I'm on holiday, but I can try again in like 3-4 weeks. |
@Quaqmre To which branch did you pushed your changes? Did you tried creating the PR via https://github.com/mattermost/mattermost-plugin-gitlab/compare? |
I can still reproduce this with: In a private mattermost 5.28.1 channel (and gitlab plugin 1.3.0) I ran: Then in my git repo I ran:
Resulted in these two gitlab notifications:
|
For this issue - There may value in continuing to deliver the tag deletion events. An alternative may simply be to clarify this behavior in the help. @aaronrothschild may have some thoughts here. |
@DHaussermann @wiggin77
Please let us know if making an extra API call is a viable approach to resolving this issue and whether we should proceed in this direction.` |
@Kshitij-Katiyar Let's try and see if it works. |
It writes "New tag" also when I delete a tag.
Output looks like this when I push a new tag and when i push --delete a tag: https://paste.ee/p/xwvbQ
Issue created from a Mattermost message by
@martbhell
.The text was updated successfully, but these errors were encountered: