meta-review/: Redesign the webpage #254
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
KVGarg
force-pushed
the
redesign-metareview-webpage
branch
5 times, most recently
from
05498e6
to
8719d4e
Compare
KVGarg
force-pushed
the
redesign-metareview-webpage
branch
from
8719d4e
to
b8b1c32
Compare
KVGarg
force-pushed
the
redesign-metareview-webpage
branch
from
b8b1c32
to
c5b944e
Compare
Travis tests have failedHey @KVGarg, 1st Build./.ci/build.shTravisBuddy Request Identifier: b07e0200-ac50-11e9-a8f1-bbdc807b8a3b |
KVGarg
force-pushed
the
redesign-metareview-webpage
branch
from
c5b944e
to
90f6fee
Compare
Travis tests have failedHey @KVGarg, 1st Build./.ci/build.shpytestTravisBuddy Request Identifier: 012922e0-b366-11e9-a522-656c855f12dd |
KVGarg
force-pushed
the
redesign-metareview-webpage
branch
from
90f6fee
to
05e7d4e
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
KVGarg
force-pushed
the
redesign-metareview-webpage
branch
2 times, most recently
from
0b0ab8f
to
47ac600
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
KVGarg
force-pushed
the
redesign-metareview-webpage
branch
from
47ac600
to
a188e9d
Compare
KVGarg
force-pushed
the
redesign-metareview-webpage
branch
from
a188e9d
to
c2f5c96
Compare
|
Do not close GitLab issues from here as mentioned earlier. |
li-boxuan
requested changes
Aug 8, 2019
templates/meta_review.html
Outdated
| </div> | ||
| <div class="metareviewer-details"> | ||
| <div class="contributor-details-part-1 column-flex"> | ||
| <a class="bold-text large-font" href="//github.com |
There was a problem hiding this comment.
I will check if it is feasible to make it case insensitive!
And will also check why the link the isn't working as it is an anchor tag
There was a problem hiding this comment.
Made search case insensitive and the link too now is working 👍
The redesigned web-page provides a nice UI/UX design to the web-page with "search" feature. Closes coala#259
KVGarg
force-pushed
the
redesign-metareview-webpage
branch
from
c2f5c96
to
cafc7e6
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The redesigned web-page provides a nice
UI/UX design to the web-page with "search"
feature.
Closes #259