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

method, qual data analysis #609

Merged
merged 7 commits into from
Sep 23, 2024
Merged

method, qual data analysis #609

merged 7 commits into from
Sep 23, 2024

Conversation

a-chappelear
Copy link
Contributor

@a-chappelear a-chappelear commented Jun 18, 2024

Changes proposed in this pull request:

Adding Qual data method per tlc ticket #408

security considerations

[Note the any security considerations here, or make note of why there are none]

None

@a-chappelear a-chappelear requested review from juliaklindpaintner and removed request for MelissaBraxton June 18, 2024 17:00
@a-chappelear a-chappelear self-assigned this Jun 18, 2024
Copy link
Member

@juliaklindpaintner juliaklindpaintner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there! I noted a few elements that I think should be changed for consistency before publishing.

I have an overarching question that really is more in relation to the original ticket for content creation, so since that has already been approved, I don't think it has to be a blocker to completing the implementation ticket. But it seems to me that the content here is too high-level to be helpful and very close to the Affinity mapping method. The original ticket was meant to provide instructions for "coding qualitative research data," which I don't think this card really does. I'll tag in @bpdesigns and @MelissaBraxton to weigh in on whether these concerns need to be addressed before publishing this new method card.

content/methods/decide/qualitative-data-analysis.md Outdated Show resolved Hide resolved
content/methods/decide/qualitative-data-analysis.md Outdated Show resolved Hide resolved
content/methods/decide/qualitative-data-analysis.md Outdated Show resolved Hide resolved
content/methods/decide/qualitative-data-analysis.md Outdated Show resolved Hide resolved
content/methods/decide/qualitative-data-analysis.md Outdated Show resolved Hide resolved
content/methods/decide/qualitative-data-analysis.md Outdated Show resolved Hide resolved
@MelissaBraxton
Copy link
Contributor

I agree with Julia's comment that I think we can add a little more detail to make this more actionable. As is, it reads a lot like affinity mapping. I'd encourage folks to start by reading through Boon's great, brief, yet practical guide on coding qual data. I think we could improve the helpfulness of this card if we include some of the details re: coding data, sorting/sifting, recoding to draw out themes.

@bpdesigns
Copy link
Member

Thank you @juliaklindpaintner and @MelissaBraxton for your review!

@a-chappelear we will need to update the content on this ticket in light of these reviews. Would you like to take on the content changes for this ticket or just own the implementation?

@a-chappelear
Copy link
Contributor Author

a-chappelear commented Jun 18, 2024

@bpdesigns I would move this ticket as blocked and re-open the content ticket for it (I agree, it seems similar to the affinity card, many items in that category could be considered a form of qual data analysis...maybe it belongs in a diff category) I'm happy to adjust, & perhaps consult with Boon and/or IA folks if available.

@a-chappelear a-chappelear requested a review from a team as a code owner July 15, 2024 18:27
@a-chappelear
Copy link
Contributor Author

PR pushed, with reworked content per https://github.com/orgs/18F/projects/41/views/1?pane=issue&itemId=23668884

@MelissaBraxton
Copy link
Contributor

@a-chappelear - I'm concerned about the mention of specific proprietary tools (Airtable, Excel) as well as links under resources to for-profit sites. We've been told by OGC that we can't appear to be endorsing products/companies, so I think all refs to proprietary stuff needs to be removed or replaced. We could revise the content under "why" to just say something about spreadsheet tools...

@a-chappelear
Copy link
Contributor Author

Good points, updated that copy @MelissaBraxton

Copy link
Contributor

@MelissaBraxton MelissaBraxton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@a-chappelear
Copy link
Contributor Author

This is ready - but it seems like it needs an approve from @juliaklindpaintner due to comments/resolved.

@edwintorres
Copy link
Member

edwintorres commented Sep 16, 2024

@amandacostello I solved the conflict

@edwintorres
Copy link
Member

Hi @juliaklindpaintner

Could you please review your review request to see if it’s done or dismiss it if it’s no longer needed?

@juliaklindpaintner juliaklindpaintner dismissed their stale review September 23, 2024 14:47

I am confident that this has been addressed!

@edwintorres edwintorres merged commit 68dc2c1 into main Sep 23, 2024
8 checks passed
@edwintorres edwintorres deleted the tlc-methods-qual-data branch September 23, 2024 20:07
@nateborr nateborr removed the blocked label Sep 26, 2024
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.

8 participants