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

feat(User): create User class #1606

Merged
merged 1 commit into from
Nov 5, 2021
Merged

Conversation

jefflembeck
Copy link
Contributor

This is a User model. It has some attributes on it that are important to
understand about the current user as we build web pages, such as their
email, if they are an admin, and whether or not they are a paid user.

Have added this user as $current_user to the exported common.inc file so
it will be referenced in any page that is used

@scottjehl
Copy link
Contributor

Well this looks like 🔥 @jefflembeck . Looking forward to looking it over tomorrow

This is a User model. It has some attributes on it that are important to
understand about the current user as we build web pages, such as their
email, if they are an admin, and whether or not they are a paid user.

Have added this user as $current_user to the exported common.inc file so
it will be referenced in any page that is used
www/common.inc Show resolved Hide resolved
@tkadlec tkadlec self-requested a review November 4, 2021 14:04
Copy link
Contributor

@tkadlec tkadlec left a comment

Choose a reason for hiding this comment

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

LGTM! Nice work @jefflembeck!

@jefflembeck jefflembeck merged commit 808c7e8 into master Nov 5, 2021
@jefflembeck jefflembeck deleted the check-user-on-every-page branch November 5, 2021 16:19
@jefflembeck jefflembeck restored the check-user-on-every-page branch November 5, 2021 19:13
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

3 participants