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

Import Column Properties #206

Closed
Stephen-Gates opened this issue Nov 8, 2017 · 4 comments
Closed

Import Column Properties #206

Stephen-Gates opened this issue Nov 8, 2017 · 4 comments
Labels
est:Moderate Moderate effort to implement est:Score=4 Score for estimate of effort required (scale of 1 upwards) f:Feature-request This issue is a request for a new feature fn:Set-Properties support:Approved Approved to be done under the support agreement support This issue is a candidate to complete under the support agreement

Comments

@Stephen-Gates
Copy link
Contributor

Stephen-Gates commented Nov 8, 2017

Desired Behaviour (for feature requests only)

As described at https://relishapp.com/odi-australia/data-curator/docs/backlog/import-column-properties

@Stephen-Gates Stephen-Gates added the f:Feature-request This issue is a request for a new feature label Nov 8, 2017
@Stephen-Gates Stephen-Gates added this to the Backlog milestone Nov 8, 2017
@Stephen-Gates Stephen-Gates modified the milestones: Backlog, v1.x.x Dec 1, 2017
@Stephen-Gates
Copy link
Contributor Author

Stephen-Gates commented Dec 13, 2017

A possible approach to providing a library of schema snippets https://github.com/Stephen-Gates/table-schema-snippets

GitHub
table-schema-snippets - A collection of field descriptions and constraints to help build a Frictionless Data table schema quickly

@Stephen-Gates
Copy link
Contributor Author

May not be needed if implement #852

@ghost
Copy link

ghost commented Jun 19, 2019

Hi @markwheels
Split out from #852, to do the work required for Column Properties import, as I want to make sure that I've understood correctly.

My question is what happens when the number of columns in the CSV and Table Schema are not equal? We may need some extra validation because at the moment these can't be different.
I presume you're talking about the column properties imported?

Thinking yes that's the simplest way to go (unless I've misunderstood this?)

  • make it strict by ensuring current table and import both match

@ghost ghost added the est:Moderate Moderate effort to implement label Jun 19, 2019
@ghost ghost modified the milestones: Backlog, Support release Jun 19, 2019
@ghost ghost added support This issue is a candidate to complete under the support agreement est:Score=4 Score for estimate of effort required (scale of 1 upwards) labels Jun 19, 2019
@markwheels markwheels added the support:Approved Approved to be done under the support agreement label Jun 19, 2019
@markwheels
Copy link

Thank @mattRedBox and @Stephen-Gates , I have added the 'support:Approved' label.

ghost pushed a commit that referenced this issue Jun 28, 2019
ghost pushed a commit that referenced this issue Jun 28, 2019
@ghost ghost closed this as completed in 18ea670 Jun 30, 2019
ghost pushed a commit that referenced this issue Jun 30, 2019
… to lock. Separated logic from main's url to frictionless for easier re-use. Added submenu items for both file and url. Improved style to suit open. Allowed title to change. Closes #742. Closes #731. Added Import Column Properties to file disable/enable list. Closes #924: Enabled difference between case-insensitive (with diacritics) and case-sensitive (without diacritics).

Signed-off-by: Matthew Mulholland <matt@redboxresearchdata.com.au>
ghost pushed a commit that referenced this issue Jun 30, 2019
… to lock. Separated logic from main's url to frictionless for easier re-use. Added submenu items for both file and url. Improved style to suit open. Allowed title to change. Closes #742. Closes #731. Added Import Column Properties to file disable/enable list. Closes #924: Enabled difference between case-insensitive (with diacritics) and case-sensitive (without diacritics).
ghost pushed a commit that referenced this issue Jul 1, 2019
ghost pushed a commit that referenced this issue Jul 1, 2019
ghost pushed a commit that referenced this issue Jul 1, 2019
ghost pushed a commit that referenced this issue Jul 1, 2019
ghost pushed a commit that referenced this issue Jul 1, 2019
…se. Added file open for resources schema and piped to new frictionless logic in place for url.
ghost pushed a commit that referenced this issue Jul 1, 2019
…e to suit open. Allowed title to change. Closes #742. Closes #731.
@ghost ghost mentioned this issue Jul 2, 2019
ghost pushed a commit that referenced this issue Apr 21, 2021
ghost pushed a commit that referenced this issue Apr 21, 2021
ghost pushed a commit that referenced this issue Apr 21, 2021
ghost pushed a commit that referenced this issue Apr 21, 2021
ghost pushed a commit that referenced this issue Apr 21, 2021
ghost pushed a commit that referenced this issue Apr 21, 2021
ghost pushed a commit that referenced this issue Apr 21, 2021
…se. Added file open for resources schema and piped to new frictionless logic in place for url.
ghost pushed a commit that referenced this issue Apr 21, 2021
…e to suit open. Allowed title to change. Closes #742. Closes #731.
ghost pushed a commit that referenced this issue Apr 21, 2021
… to lock. Separated logic from main's url to frictionless for easier re-use. Added submenu items for both file and url. Improved style to suit open. Allowed title to change. Closes #742. Closes #731. Added Import Column Properties to file disable/enable list. Closes #924: Enabled difference between case-insensitive (with diacritics) and case-sensitive (without diacritics).
ghost pushed a commit that referenced this issue Apr 21, 2021
… to lock. Separated logic from main's url to frictionless for easier re-use. Added submenu items for both file and url. Improved style to suit open. Allowed title to change. Closes #742. Closes #731. Added Import Column Properties to file disable/enable list. Closes #924: Enabled difference between case-insensitive (with diacritics) and case-sensitive (without diacritics).

Signed-off-by: Matthew Mulholland <matt@redboxresearchdata.com.au>
ghost pushed a commit that referenced this issue Apr 21, 2021
… to lock. Separated logic from main's url to frictionless for easier re-use. Added submenu items for both file and url. Improved style to suit open. Allowed title to change. Closes #742. Closes #731. Added Import Column Properties to file disable/enable list. Closes #924: Enabled difference between case-insensitive (with diacritics) and case-sensitive (without diacritics).
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
est:Moderate Moderate effort to implement est:Score=4 Score for estimate of effort required (scale of 1 upwards) f:Feature-request This issue is a request for a new feature fn:Set-Properties support:Approved Approved to be done under the support agreement support This issue is a candidate to complete under the support agreement
Projects
None yet
Development

No branches or pull requests

2 participants