-
Notifications
You must be signed in to change notification settings - Fork 45
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
gamification/: Redesign the webpage #261
base: master
Are you sure you want to change the base?
Conversation
0bde2e1
to
7801684
Compare
Travis tests have failedHey @KVGarg, 1st Buildcoala --non-interactive -V
TravisBuddy Request Identifier: e8eef220-8f9a-11e9-b69a-57a176dcbff9 |
7801684
to
80147e4
Compare
80147e4
to
2a5a4c8
Compare
2a5a4c8
to
7deac4c
Compare
7deac4c
to
3a75b5a
Compare
Travis tests have failedHey @KVGarg, 1st Build./.ci/build.sh
TravisBuddy Request Identifier: b02a63c0-ac50-11e9-a8f1-bbdc807b8a3b |
3a75b5a
to
c5e079d
Compare
Travis tests have failedHey @KVGarg, 1st Build./.ci/build.sh
TravisBuddy Request Identifier: c60197a0-b366-11e9-a522-656c855f12dd |
c5e079d
to
807b062
Compare
The updation in model adds new data fields which will be useful, in further work. Also, it will reduce the API calls, which are being made to fetch these information. Instead of making new API Calls, we can get them from webservices and use them directly.
The openhub module got disabled due to some reasons in community project last year. And, currently the module is raising the CI errors which need to be avoided. They can be worked on once we're ready to enable the openhub module once again in this project in future.
As in future, the data will be fetched from coala-webservices intead of @sks444 repository (webservices.coala.io). So, to remove that dependency the tests have been modified accordingly.
Running command ".ci/build.sh" locally on pc used to failed the build, if there is more than one git remote's. Fixes coala#235
807b062
to
7313215
Compare
The enhancement includes addition of materialize css, JQuery, responsiveness, and easy-navigation of website features. The easy-navigatibility is achieved by adding a navbar with display of meta -review and gamification leaderboard on homepage. Apart from this, the activity graph url is omitted from website by displaying the graph itslef on the homepage on large devices. Closes coala#255
The newly created webpage combines the previous two webpages- info.txt and log/index.html. This web-page combines the results of both the pages and shows them in a better UI/UX with additional features of filtering and searching within the existing logs. The logs are fetched from a JSON file which is created from the logs stored in the log file _site/community.log Closes coala#256
The redesigned web-page displays the data in a better UI/UX form with some additional information to make it more interactive and attractive. Closes coala#257
The redesigned web-page provides a nice UI/UX design to the web-page with "search" feature. Closes coala#259
7313215
to
686c2d3
Compare
@sks444 Should I add filters to gamification leaderboard or is it OK, like now we currently have ? |
@KVGarg, What Kind of filter do you want to add? |
Filter results based on score range, badges earned, and the level! |
Sounds good, add it. |
035e973
to
8a510c6
Compare
db0954e
to
de8d3ab
Compare
The redesigned webpages provides a enhanced UI/UX design to web-page with additional functionality of searching the contributors. Closes coala#260
de8d3ab
to
f581b43
Compare
Added Filters 😇 Can you check the corresponding UI for it and do some random tests whether they are working or not? |
The redesigned webpages provides a enhanced
UI/UX design to web-page with additional
functionality of searching the contributors.
Closes #260