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

Remove journey boxes from docs' landing page #4982

Closed
Savio-Sou opened this issue May 6, 2024 · 1 comment · Fixed by #5400
Closed

Remove journey boxes from docs' landing page #4982

Savio-Sou opened this issue May 6, 2024 · 1 comment · Fixed by #5400
Labels
documentation Improvements or additions to documentation enhancement New feature or request

Comments

@Savio-Sou
Copy link
Collaborator

Savio-Sou commented May 6, 2024

Problem

https://noir-lang.org/ current lands on the index.jsx page with boxes for readers to self-categorize their interests and backgrounds:

image

@noir-lang/developerrelations brought up the self-categorization could hurt DocEx instead of improving it though, leaving readers confused and frustrated as their understanding of the categories might not necessarily match how doc authors comprehend them.

Happy Case

Remove the self-categorization boxes from index.jsx.

Project Impact

Nice-to-have

Impact Context

Improve docs reading experience.

Workaround

Yes

Workaround Description

Promote https://noir-lang.org/docs/ instead of https://noir-lang.org/.

Additional Context

The index.jsx page was introduced in #4081.

Would you like to submit a PR for this Issue?

None

Support Needs

No response

@Savio-Sou Savio-Sou added enhancement New feature or request documentation Improvements or additions to documentation labels May 6, 2024
@Savio-Sou Savio-Sou added this to Noir May 6, 2024
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir May 6, 2024
@Savio-Sou Savio-Sou changed the title Remove index.jsx from docs and redirect landing to /docs Remove journey boxes from docs' landing page Jul 3, 2024
@Savio-Sou
Copy link
Collaborator Author

Discussed further with @noir-lang/developerrelations: instead of removing the landing page, we should remove the boxes for this Issue (and update content on the landing page where necessary with further PRs).

Issue updated accordingly.

github-merge-queue bot pushed a commit that referenced this issue Jul 3, 2024
# Description

## Problem\*

Resolves #4982

## Summary\*

These persona boxes were a rudimentary way to get some info about who
was coming to the docs. We have proper analytics now, so we can remove
them.


## Additional Context



## Documentation\*

Check one:
- [ ] No documentation needed.
- [ ] Documentation included in this PR.
- [ ] **[For Experimental Features]** Documentation to be submitted in a
separate PR.

# PR Checklist\*

- [ ] I have tested the changes locally.
- [ ] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Noir Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
Archived in project
1 participant