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

My teachers and tabbed buckets #5

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

GrahamQuince
Copy link
Contributor

Two new snippets.

Two snippets:

Site icon displays a site's icon as its image

Assignment List lists out the assignments for a student without them
accessing the app
Site icon as image & Assignment list
and improved site icon code based on feedback
Added correct code for assignments
Site icon is commented, assignments list rewritten for student accessing
latest assignments
Updated to tidy up styling and add comments
@Smudge3806 Smudge3806 self-assigned this Apr 7, 2016
Added some a third snipped based on a school's request

/// Get all the groups assoicated with logged in user
Frog.Model.api('users.getDataInCategory', {
user_uuid:user.uuid,
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we not using .attr().

user_uuid: user.attr('uuid')

@Smudge3806
Copy link
Contributor

Formatting is inconsistent and incorrect in most places. Grammar (especially comments) is incorrect in several files. The diff also contains the assignment-list snippet which was merged last week.

Please ensure your folk is up-to-date with frogeducation/frogos-snippets:master as there are some fixes in there that will not be in the repo.

@Smudge3806
Copy link
Contributor

I think we need to review how you use GIT. The desktop application does hide some of the complexities but it also prescribes you to a workflow that is fairly restrictive and confusing. I think you will understand the CLI and it will allow for easier management. Let me know if you are comfortable with this and I can set something up to go through it with you

@GrahamQuince
Copy link
Contributor Author

Hi Chris,

Thank you. And yes to all your suggestions.

Are you free this afternoon? Or anytime this week?

Graham

From: Chris Smith [mailto:[email protected]]
Sent: 18 April 2016 09:25
To: frogeducation/frogos-snippets [email protected]
Cc: Graham Quince [email protected]
Subject: Re: [frogeducation/frogos-snippets] My teachers and tabbed buckets (#5)

I think we need to review how you use GIT. The desktop application does hide some of the complexities but it also prescribes you to a workflow that is fairly restrictive and confusing. I think you will understand the CLI and it will allow for easier management. Let me know if you are comfortable with this and I can set something up to go through it with you


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com//pull/5#issuecomment-211263738

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.

2 participants