Skip to content

For gem edit pages, build a new linkset if gem is missing one#1844

Closed
kerrizor wants to merge 1 commit into
rubygems:masterfrom
kerrizor:kerrizor/build-new-linkset-if-missing-for-gem-edit-pages
Closed

For gem edit pages, build a new linkset if gem is missing one#1844
kerrizor wants to merge 1 commit into
rubygems:masterfrom
kerrizor:kerrizor/build-new-linkset-if-missing-for-gem-edit-pages

Conversation

@kerrizor
Copy link
Copy Markdown
Contributor

@kerrizor kerrizor commented Dec 8, 2018

We have a few places where we work around the issue of a gem not having
an associated linkset record, so this at least is in that genre of
change.

This was reported to us via the support site; something about that
user's gemspec is funky, and a linkset isn't being created. The unusual
thing is that this user attempted to edit via the web UI, and I wouldn't
be surprised if this is a low-traffic action, which would explain why we
wouldn't have seen this error crop up previously.

http://help.rubygems.org/discussions/problems/34216-unable-to-edit-newly-published-gem-server-error

We have a few places where we work around the issue of a gem not having
an associated linkset record, so this at least is in that genre of
change.

This was reported to us via the support site; something about that
user's gemspec is funky, and a linkset isn't being created. The unusual
thing is that this user attempted to edit via the web UI, and I wouldn't
be surprised if this is a low-traffic action, which would explain why we
wouldn't have seen this error crop up previously.

http://help.rubygems.org/discussions/problems/34216-unable-to-edit-newly-published-gem-server-error
@kerrizor kerrizor requested a review from sonalkr132 December 8, 2018 18:03
@kerrizor
Copy link
Copy Markdown
Contributor Author

kerrizor commented Dec 8, 2018

@sonalkr132 and I chatted and we're not going to merge this at this time, as the entire linkset form is going away

@kerrizor kerrizor closed this Dec 8, 2018
@kerrizor kerrizor deleted the kerrizor/build-new-linkset-if-missing-for-gem-edit-pages branch December 8, 2018 18:51
@sonalkr132
Copy link
Copy Markdown
Member

See #1815

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants