-
Notifications
You must be signed in to change notification settings - Fork 13
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
collections: allow a collection to be set/mapped from a spreadsheet #902
Comments
I'd love some help with this and it's a nice stand-alone challenge - but the spec is super rough right now and there are no links to docs to get a new dev started. If you're reading this and you'd like to help, this is a great little project. Please tag me here and I'll help you get going! |
@SatyamMattoo Are you still looking for projects to contribute to? |
@josephjclark Sure! I would love to contribute! Do you have some deadline planned for this? I might take some time to complete this due a busy schedule. |
Hi @SatyamMattoo , nice to hear from you! No deadline as yet. No pressure either - I just thought it would be a fun one for you :) I'll leave this opened and unassigned for now. Drop me a line here if/when you think you have time. Best to focus a few days on it. We'll set up a call and I'll talk you through it. You'll have to get up to speed with our new Collections feature - but after that I think it's quite straightforward, standalone and valuable. |
@josephjclark Thanks for considering me! I'll take some time this weekend to review the Collections feature, and we can discuss it on Monday if that works for you? |
@SatyamMattoo Sure but only if you have time to commit to the work next week. Book a slot in my calender here and we'll talk about it on Monday |
endpoint: collections CLI docs: https://docs.openfn.org/documentation/collections-cli#uploading-items |
I want to:
This feels like it's something the CLI is very capable of doing.
If the sheet just has two columns, we can just pull them as key/value pairs. Otherwise we need to take a range (or I suppose two columns). We may need to know whether the first two rows are a header and can be ignored (although it's probably harmless to include a header in the map - it's just one redundant key)
The mapping should be a single JSON object uploaded as a key to a single value
So the CLI command is like:
Maybe we need extra CLI args for "parse items as spreadsheet" or "choose this range" or "ignore headers".
The text was updated successfully, but these errors were encountered: