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

Update 02-working-with-openrefine.md #82

Merged
merged 1 commit into from
Jul 2, 2021

Conversation

K05lowi
Copy link
Contributor

@K05lowi K05lowi commented Dec 3, 2020

line 229 remove 'respodent_wall_type'
The reference to the column respondent_wall_type must misplaced/an error and learners will find the reference confusing.

Instructions

Thanks for contributing! ❤️

If this contribution is for instructor training, please email the link to this contribution to
[email protected] so we can record your progress. You've completed your contribution
step for instructor checkout by submitting this contribution!

Keep in mind that lesson maintainers are volunteers and it may take them some time to
respond to your contribution. Although not all contributions can be incorporated into the lesson
materials, we appreciate your time and effort to improve the curriculum. If you have any questions
about the lesson maintenance process or would like to volunteer your time as a contribution
reviewer, please contact The Carpentries Team at [email protected].

You may delete these instructions from your comment.

- The Carpentries

line  229 remove  'respodent_wall_type'
@K05lowi
Copy link
Contributor Author

K05lowi commented Dec 3, 2020

This suggested deletion of content line 229 is part of the instructor training checkout process. I know the lesson is already packed, but as the dataset is so big please consider introducing the drop down menu under 'All' and introduce how to reorder the rows. I'll happily help with this addition.

@K05lowi K05lowi closed this Dec 3, 2020
@brownsarahm brownsarahm reopened this Dec 3, 2020
@brownsarahm
Copy link
Contributor

HI @K05lowi, this looks like a great fix to the lesson material. Send the link to this to [email protected] to get credit for your contribution and I'ld like to merge it in, unless you had a bigger reason to close this pull request?

@bencomp
Copy link
Contributor

bencomp commented Jul 2, 2021

The change in this PR has been implemented in the already merged #91 so this PR could be closed.

@brownsarahm brownsarahm merged commit ec63dff into datacarpentry:gh-pages Jul 2, 2021
zkamvar pushed a commit that referenced this pull request Apr 28, 2023
Update 02-working-with-openrefine.md
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.

3 participants