Linking to specific documentation on main dashboard pages#825
Linking to specific documentation on main dashboard pages#825
Conversation
Made-with: Cursor
|
Visit the preview URL for this PR (updated for commit 1b32125): https://hs-levante-admin-dev--pr825-linking-to-docs-3yg43sdx.web.app (expires Mon, 23 Mar 2026 18:59:56 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 7889bff1da3bcc333d7422b9fc863c65b3962be7 |
fhconte
left a comment
fhconte
left a comment
There was a problem hiding this comment.
Just sent some really small grammar and html default pattern correction as a common security practice. Non-blocking for this PR.
| The site to which users are added is now determined by the site selected in the site selector (top right). | ||
| The Add Users process will only add users to the currently selected site, and will display a warning if your file contains a site column with values that do not match the currently selected site. | ||
| Users in those rows will cause the add users process to fail. We recommend splitting up your user files by site. | ||
| Previously, <code>site</code> was a requiired column in the user information file. Now it is no longer required, because all users will be added to the site chosen within the site selector on the top right of the dashbaord. If <code>site</code> exists in your spreadsheet and its values do not match the selected site, you will see a warning. We recommend having a separate user information file for each site. |
There was a problem hiding this comment.
Just a grammar correction for 'required' and 'dashboard'
| <p>Notes:</p> | ||
| <ul> | ||
| <li>Reminder: Users either belong to a School and Class OR a Cohort.</li> | ||
| <li>Complete the appropriate column(s) according to the users' Group membership (School and Class OR Cohort).</li> |
There was a problem hiding this comment.
Found a duplicate space in 'according to'
|
|
||
| <div class="how-to-section mb-4"> | ||
| <h3>How to create groups</h3> | ||
| <div class="text-md text-gray-500 mb-1 line-height-3">Users are added to groups in order to receive tasks and surveys through assignments. Which kinds of groups you need will depend on your study design. Before getting started, please read <a href="https://researcher.levante-network.org/dashboard/create-a-group" target="_blank">the documentation on creating groups.</a></div> |
There was a problem hiding this comment.
Follow-up: Consider adding rel="noopener noreferrer" to the external target="_blank" links for consistency and common security practice. Non-blocking for this PR.
| </div> | ||
| <div class="file-requirements-section"> | ||
| <h2>User Information File Requirements</h2> | ||
| <p>Formatting requirements for your user information file are detailed in the table below. For more information, refer to the <a href="https://researcher.levante-network.org/dashboard/add-users" target="_blank">add users documentation</a>.</p> |
There was a problem hiding this comment.
Same follow-up: Consider adding rel="noopener noreferrer" to the external target="_blank".
fhconte
left a comment
fhconte
left a comment
There was a problem hiding this comment.
I will wait for the grammar corrections and functionalitty tests to confirm its approval
fhconte
dismissed
their stale review
I will wait for the grammar corrections and functionalitty tests to confirm its approval
2 participants
Proposed changes
Types of changes
What types of changes does this pull request introduce?
Additional Notes