Proper message when registering an already registered snap #623

Merged
merged 8 commits into from Jul 4, 2016

Conversation

Projects
None yet
2 participants
Collaborator

sergiusens commented Jul 1, 2016

LP: #1596782

Signed-off-by: Sergio Schvezov sergio.schvezov@ubuntu.com

sergiusens added some commits Jul 1, 2016

Proper message when registering an already registered snap
LP: #1596782

Signed-off-by: Sergio Schvezov <sergio.schvezov@ubuntu.com>
Member

elopio commented Jul 4, 2016

lgtm. Just a small detail, the error I get when I try to register a name twice is:

The name 'u1test20160702' is already taken.

We can if needed rename snaps to ensure they match the expectations of most users. If you are the publisher most users expect for 'u1test20160702' then claim the name at 'https://myapps.developer.ubuntu.com/dev/click-apps/register-name/?series=16&name=u1test20160702'

That is not the URL to claim the name. That is to register another name, with a link to fill a form for claiming. So the message we are giving is a little inaccurate. This message would be accurate if the link is to the claim form: https://myapps.developer.ubuntu.com/dev/click-apps/register-name-dispute/?series=16&name=u1test20160702

I think the store should return this dispute url instead.

Collaborator

sergiusens commented Jul 4, 2016

@elopio I went with constructing one as the error message is pretty lame if not.

Member

elopio commented Jul 4, 2016

great, with a link to the bug. 👍

sergiusens added some commits Jul 4, 2016

Member

elopio commented Jul 4, 2016

Ran the tests with fake and staging servers. All good.
But I am not authorized to merge this, @sergiusens you go.

@sergiusens sergiusens merged commit fca736b into snapcore:master Jul 4, 2016

3 of 4 checks passed

Examples tests
Details
autopkgtest Success
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.0001%) to 96.253%
Details

@sergiusens sergiusens deleted the sergiusens:bugfix/1596782/register-already-registered branch Jul 4, 2016

kalikiana pushed a commit to kalikiana/snapcraft that referenced this pull request Apr 6, 2017

Proper message when registering an already registered snap (#623)
LP: #1596782

Signed-off-by: Sergio Schvezov <sergio.schvezov@ubuntu.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment