hypothesis / hypothesis/lms

Define a new property in LTIUser to mean "has_valid_role"

Open
#6,055 0 comments 0 reactions 0 assignees View on GitHub
Good introductory issue
Dominant language
Python
Stars
53
Forks
16
Avg merge
14d 5h
Merged PRs (30d)
14

Description

Maybe we could define a new property that wraps and internally calls these three, so that we can do `if lti_user.has_valid_role`, or something in those lines.

Just something to consider in future.

_Originally posted by @acelaya in https://github.com/hypothesis/lms/pull/6052#discussion_r1495557661_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.