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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Notebooks For All usability test results #26

Merged
merged 1 commit into from Aug 8, 2023

Conversation

isabela-pf
Copy link
Contributor

I'm excited to bring some accessibility efforts to the Jupyter surveys repo!

This PR:

  • Creates a directory under surveys for the Notebooks For All project (accessibility testing and fixes for HTML converted notebook files and nbconvert)
  • Adds a project readme that provides information about the tests. It also links to the results and resources.
  • Adds the project to the root repo readme under Jupyter user surveys

Please let me know if you have any questions. Thanks in advance for review! 馃尰

cc: @eteq, @Jenneh, @smythp, and @tonyfast because your names are mentioned as the project team.

@isabela-pf
Copy link
Contributor Author

I know this repository is low traffic, but I want to make some noise to ping this for review again if possible. Please let me know if there's anything I can do to make review easier. Thank you! 馃尰

Copy link

@tonyfast tonyfast left a comment

Choose a reason for hiding this comment

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

i've looked through and im super excited to see these results be upstreamed. i can't merge, but i'll sign off on a review.

@tonyfast
Copy link

tonyfast commented Aug 7, 2023

@Carreau @Zsailer any chance yall could merge this?

@Zsailer
Copy link
Member

Zsailer commented Aug 7, 2023

Thanks, @isabela-pf! This looks awesome. Great work!

Unfortunately, I don't have merge rights on this repo anymore. I think I did at one point, but I was probably dropped when we refactored the governance structure.

@Carreau Carreau merged commit 495505c into jupyter:master Aug 8, 2023
@Carreau
Copy link
Member

Carreau commented Aug 8, 2023

Boom, merged.

@isabela-pf
Copy link
Contributor Author

Thank you!

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.

None yet

4 participants