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

Make resource name default to file name #1372

Closed
mwainwright opened this issue Dec 9, 2013 · 5 comments

Comments

Projects
None yet
8 participants
@mwainwright
Copy link

commented Dec 9, 2013

Users are usually not sure what to enter as the 'resource name' and it ends up being blank. It's mildly helpful for a resource to have a name, however.

I suggest the field should be pre-populated with the filename, or (for a linked file) component of the name after the last "/". E.g. for http://example.com/this/that/theother.csv the "name" would be "theother.csv", unless the user changed it.

Obviously, if the URL ends with a / like http://example.com/this/that/theother/ the default name should be "theother", rather than being blank.

@rossjones

This comment has been minimized.

Copy link
Collaborator

commented Feb 12, 2016

@smotornyuk is this fixed now?

@wardi

This comment has been minimized.

Copy link
Contributor

commented Feb 12, 2016

No, there's a PR to address this with client side JS, but it overwrites the name even if it has been given so I haven't merged it

@rossjones

This comment has been minimized.

Copy link
Collaborator

commented Feb 12, 2016

Will assign to @smotornyuk then in case he wants to finish it off.

smotornyuk added a commit to smotornyuk/ckan that referenced this issue Feb 16, 2016

[ckan#1372] Make resource name default to file name
Currently user have to manually add resource name after or before
uploading/linking resource

After this change name will be automatically filled when resource
uploaded or added link to resource, but only if name has not been
previously modified by user and was empty from the very begining.
i.e. this won't work for resource update
@smotornyuk

This comment has been minimized.

Copy link
Member

commented Feb 16, 2016

@rossjones , i've fixed problem mentioned by Ian and updated pull request.

smotornyuk added a commit to smotornyuk/ckan that referenced this issue Mar 9, 2016

@TkTech TkTech closed this May 10, 2016

@davidread

This comment has been minimized.

Copy link
Contributor

commented May 24, 2016

This issue was closed due to inactivity. Feel free to reopen if you have more feedback or are interested it working on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.