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

Hyperlink to users in a few more places #3205

Merged
merged 3 commits into from
Feb 20, 2021
Merged

Hyperlink to users in a few more places #3205

merged 3 commits into from
Feb 20, 2021

Conversation

kkbrum
Copy link
Contributor

@kkbrum kkbrum commented Feb 12, 2021

Fixes #1374.

This adds user links to the scheduler, the finaid approval module, and the manage events pages. The only other place I noticed that could use links that don't currently exist is on the scheduling checks page, but I wasn't able to add them easily so am leaving it be for now.

@lgtm-com
Copy link

lgtm-com bot commented Feb 12, 2021

This pull request introduces 1 alert when merging 06b7f08 into 04f87d1 - view on LGTM.com

new alerts:

  • 1 for Potentially unsafe external link

@lgtm-com
Copy link

lgtm-com bot commented Feb 20, 2021

This pull request introduces 1 alert when merging 841cca1 into f353bf8 - view on LGTM.com

new alerts:

  • 1 for Potentially unsafe external link

@hwatheod hwatheod merged commit 64c8e16 into main Feb 20, 2021
@hwatheod hwatheod deleted the hyperlink-users-new branch February 20, 2021 20:28
willgearty added a commit that referenced this pull request Feb 26, 2021
willgearty added a commit that referenced this pull request May 27, 2021
* Initial docs for stable release 13

* Docs for #3116, #3117, and #3118

* Added docs about django upgrade

* Docs for #3128

* Docs for #3129, #3133, #3134, and #3137

* Docs for #3156 and #3153

* Docs for #3174, #3163, and #3184

* Docs for #3139, #3140, and #3141

* Docs for #3143, #3150, #3154, #3160, #3162, and #3168

* Docs for #3171, #3185, #3186, and #3188

* Docs for #3131 and #3189

* Docs for #3149 and #3190

* Docs for #3193, #3194, #3195, #196, and #3197

* Clarification

* Docs for #3192, #3201, #3209, and #2248

* Docs for #3204, #3212, #3214, #3205, 9fd073c, and #3226

* Docs for #3232, de5861c, #3231, #3233, #3234, #3237, #3238, and #3239

* Fix indent

* Docs for #3227 and #3235

* Add missing word

* spelling

* Docs for e57581f, #3255, #3253, #3257, and #3249

* Docs for #3254, #3260, and #3262

* Docs for #3263, #3272, #3240, #3264, #3266, and #3270

* clarifications

* Docs for #3283 and #3252

* Docs for #3288 and misc commits

* Docs for #3292, #3311, #3286, #3289, and #3279

* Docs for a377f0d; move note

* Docs for #3315, #3290, and #3322

* Docs for #3273 and #3317

* Final edits
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

All teacher, student names in manage pages should hyperlink to user view
3 participants