-
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
data/: Redesign the contributors webpage #253
base: master
Are you sure you want to change the base?
Conversation
Travis tests have failedHey @KVGarg, TravisBuddy Request Identifier: 746e01e0-8e5d-11e9-8722-2b49c01fe0f5 |
27e37e1
to
294e38a
Compare
Travis tests have failedHey @KVGarg, 1st Buildcoala --non-interactive -V
TravisBuddy Request Identifier: 646047d0-8e9a-11e9-8722-2b49c01fe0f5 |
294e38a
to
8816cca
Compare
Travis tests have failedHey @KVGarg, 1st Buildcoala --non-interactive -V
TravisBuddy Request Identifier: 0f3c3180-8ea2-11e9-8722-2b49c01fe0f5 |
templates/index.html
Outdated
{% block main-content %} | ||
|
||
<div class="community-header-image"> | ||
<img src="{% static 'images/os_community_header.jpg' %}"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This image looks too large IMO.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported from Photoshop itself for web display! Adjusted the header image height to make it look good on large screen otherwise it would have taken whole visible screen to display that image.
Any suggestions? On how can I make it better.
image looks too large
468b067
to
8e5a89e
Compare
0d1747a
to
66e66c7
Compare
66e66c7
to
6abff09
Compare
Travis tests have failedHey @KVGarg, 1st Build./.ci/build.sh
TravisBuddy Request Identifier: ad90cd20-ac50-11e9-a8f1-bbdc807b8a3b |
6abff09
to
65ca0b0
Compare
Travis tests have failedHey @KVGarg, 1st Buildpytest
TravisBuddy Request Identifier: 767c9f90-b299-11e9-820a-1b236510c586 |
65ca0b0
to
fa309c3
Compare
Travis tests have failedHey @KVGarg, 1st Build./.ci/build.sh
TravisBuddy Request Identifier: 9066b630-b365-11e9-a522-656c855f12dd |
949a971
to
864e2f4
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
063c1ca
to
bcdabec
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
ed5116d
to
24193f5
Compare
24193f5
to
0746873
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
cc: @li-boxuan
Closes #240 , #231 , #141 , #139, #140