Skip to content
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

RTC Labeling Removes All Other Labels #947

Open
mbabker opened this issue Feb 17, 2017 · 14 comments
Open

RTC Labeling Removes All Other Labels #947

mbabker opened this issue Feb 17, 2017 · 14 comments
Assignees
Labels

Comments

@mbabker
Copy link
Contributor

mbabker commented Feb 17, 2017

The RTC behavior should only add/remove that specific label. Whatever is causing all other labels and the milestone to be removed needs to be corrected.

@zero-24
Copy link
Contributor

zero-24 commented Feb 17, 2017

https://github.com/joomla/jissues/blob/master/src/App/Tracker/Controller/Hooks/Listeners/JoomlacmsPullsListener.php#L330-L351

Maybe this is a permission problem? If you don't have the permissions to change the label / milestones than the labels / milestones got removed.

So i think this could be a problem in the save method where the milestones and labels are empty if you don't have the permissions to change them. This result than in removing them.

@zero-24
Copy link
Contributor

zero-24 commented Feb 25, 2017

hmm i have just noticed that the issue i'm editing does not contain any labels & milestone on the tracker

unbenannt3

But does contain some labels and milestone on github.

unbenannt2

So this looks like a bigger problem to me?

@b2z
Copy link
Member

b2z commented Mar 14, 2017

@zero-24 may be ;)
@mbabker can you please provide DB export of i.j.org to replicate the problem? AFAIK you did it something like this before in other issue. Do not remember, in which :)

@mbabker
Copy link
Contributor Author

mbabker commented Mar 14, 2017

@b2z b2z self-assigned this Mar 14, 2017
@b2z b2z added the bug label Mar 14, 2017
@b2z
Copy link
Member

b2z commented Mar 15, 2017

@mbabker thanks!

@zero-24 I see that you was able to set labels and milestone. I also tried to edit this issue and it contains labels and milestones. Is it specific case for this issue or it is happening all the time?

I do not think that the permission is the issue here as we display labels only for user with manage rights.

{% if user.check('manage') %}

And this users for sure are those who has permissions. I think the problem was that this issue did not have any label and milestone on the time of your editing. Why? That is the question :)

@zero-24
Copy link
Contributor

zero-24 commented Mar 15, 2017

I think the problem was that this issue did not have any label and milestone on the time of your editing. Why? That is the question :)

Correct. Maybe the sync job is failing for any reason?

@b2z
Copy link
Member

b2z commented Mar 15, 2017

Maybe the sync job is failing for any reason?

Hate to say it, but it seems that it is sync job again :(

@zero-24
Copy link
Contributor

zero-24 commented Mar 15, 2017

@mbabker do you have any errors or similiar in the logs?

@mbabker
Copy link
Contributor Author

mbabker commented Mar 15, 2017

All the cron logging is telling me is that weblinks and the CMS jobs aren't running in full. Not much has changed since #915 or #676 were logged.

@b2z
Copy link
Member

b2z commented Mar 24, 2017

Can we close it?

@b2z
Copy link
Member

b2z commented Aug 21, 2017

@zero-24 is issue still valid?

@zero-24
Copy link
Contributor

zero-24 commented Aug 21, 2017

Saddly yes. When you are a user with no persmissions on gihub but just on the tracker and set a item e.g. to RTC all labels expected RTC are removed.

@b2z
Copy link
Member

b2z commented Sep 8, 2017

:( I will try to replicate and fix it.

@b2z
Copy link
Member

b2z commented Nov 9, 2017

Annoying. @mbabker can you give actual DB state from issues to import? Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants