-
Notifications
You must be signed in to change notification settings - Fork 16
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
Successful trunk version deletion redirects to pod page #168
Comments
Tests imply this is known,
does this meant the |
I think I just made it redirect to the wrong thing? I'll look into this. |
Just ran into this as well when deprecating MaterialMotionDirectManipulationFamily. |
#206 did not fix it alas and was reverted |
I think what we may need to do is redirect to the push jobs API endpoint? |
that 's what we do now I think, was thinking of adding an API route for whether a pod was deleted |
I don't understand how to get rid of the error,please tell me in detail. @orta 😁 |
@ripperhe to my understanding, the issue is still open, so you can't get rid of the error. But the deletion is still a success despite the error: i.e., if you attempt to delete a second time, it will tell you it's already deleted. |
@Coeur ok, I get it. thank you! |
Is this issue still open or fixed? I am almost to the same error and opened the same issue here. |
Yeah, there was something that came up that required more work than I had time for and I've not got back to it, so the issue still exists. |
the same error |
The issue still exists. Any chance this might be fixed as it essentially breaks any scripts (expecting a success response). |
It's been an issue since 2016, so it's more likely that a script could break if someone fixes the issue. |
Actually, the deletion worked, but I got a big error message even though it worked. |
Same here, just ran |
@orta If I knew how to fix it, I would, and I would deal with the potentially broken script when/if reported. |
This is the error received:
Technically its a 404, but we're not too fussed. CocoaPods is a project from Boris Bügling, Marius Rackwitz, Orta Therox, Samuel Giddins, and The CocoaPods Dev Team with contributions from many, many others. Lovingly sponsored by Artsy, Button, Capital One, Discontinuity, Fingertips, Heroku, RubyMotion, Sauspiel, Slack, SoundCloud, Stripe and Technology Astronauts. Support |
Bump :/ |
I'm probably never going to look at this, so if there's no updates on this thread from someone else shipping a successful PR, then there's no point in bumping this thread |
@orta As someone else mentioned, if I knew how to fix it without breaking anything else, I would. That's why myself and others are asking the project team and experts to at least have a look. If you actually mean that you don't know how to fix it also, then that is a different type of response. |
No, I mean I'll probably never put the time in to look at the issue as it's generally unimportant to me and I don't want to spend my free time on it. All open source is shared cultural heritage, you don't know how to do it right now - but anyone in this thread could learn how the site works and submit a PR if they wanted. |
That is fair enough. However your point about anyone "in the thread could learn", means, to me at least, that there should be no issue with people bumping the thread to inform others that it is still an issue for people. However I would agree that there are ways to "bump" a thread :-) |
Successful pod version deletions redirect to pod page
From my terminal:
The text was updated successfully, but these errors were encountered: