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

Feature/7 basic results #22

Open
wants to merge 8 commits into
base: prototype-v0.1.0
Choose a base branch
from

Conversation

eddiejaoude
Copy link
Contributor

  • Call API
  • List results

screenshot 2016-01-06 07 10 47

@eddiejaoude eddiejaoude added this to the v0.2.*: Prototype milestone Jan 6, 2016
@@ -0,0 +1,107 @@
<!doctype html>
<html lang="en" ng-app="memberApp">

Choose a reason for hiding this comment

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

Probably better to use angular.bootstrap vs ng-app gives more flexibility when you get onto more complex coding

@andrew-cunliffe
Copy link

jQuery and Angular in a single project... better to pick a single JavaScript framework than use two, I have never found a valid need to use jQuery within an Angular project, there are libraries for everything that are alternatives to jQuery ones, also.... you can normally wrap any third party library as an Angular service and still use it (when you get to needing one that does not exist as a module you can load) checkout Angular-UI projects, there is a bootstrap piece that you can use to remove all the bootstrap jQuery, and even has some extra features that are pretty handy when working with bootstrap... or just use the Angular Material project :)

Otherwise good start!

@eddiejaoude
Copy link
Contributor Author

Thanks for comments @andrew-cunliffe, I will go through them 😄

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