CTI-website-frontend icon indicating copy to clipboard operation
CTI-website-frontend copied to clipboard

Update landing page (#909)

Open bruceplai opened this issue 2 years ago • 6 comments

Closes #909

  • Update Google Analytics measurement ID

  • Update root route to point to homepage

  • Simplify Layout

  • Update breadcrumbs to use root route

  • Update spec files to use root route

  • Remove references to old landing page

bruceplai avatar Oct 09 '21 22:10 bruceplai

On cypress/fixtures/faqs.json file you have changed the a href to "[http://civictechindex.org](http://civictechindex.org%5C)" But on FAQ page on What is Civic Tech Index? On clicking on Civic Tech Index hyperlink , it still goes to "http://civictechindex.org/home" page

@bhaggya faqs.json is for testing. If you go the FAQ page, then it currently calls the production backend API. That data returned by the API still has the old homepage URL. You can verify this by fetching the raw JSON in your browser with the following URL: https://api.civictechindex.org/api/faqs/

bruceplai avatar Oct 09 '21 23:10 bruceplai

On cypress/fixtures/faqs.json file you have changed the a href to "http://civictechindex.org" But on FAQ page on What is Civic Tech Index? On clicking on Civic Tech Index hyperlink , it still goes to "http://civictechindex.org/home" page

@bhaggya faqs.json is for testing. If you go the FAQ page, then it currently calls the production backend API. That data returned by the API still has the old homepage URL. You can verify this by fetching the raw JSON in your browser with the following URL: https://api.civictechindex.org/api/faqs/

I didnt know that ,the API needs to be changed

bhaggya avatar Oct 10 '21 00:10 bhaggya

@ladissi we will most likely need to update some data in the production db that are still referring to the /home route since that should no longer exist.

bruceplai avatar Oct 10 '21 21:10 bruceplai

This is purely code style preferences: I noticed several files are inconsistent with ordering of keys in their breadcrumb links. (Ex.)

{ href: '/', name: 'Home' } vs { name: 'Home', href: '/' }  

It could be nice to change them to one standardized style, it seems like more files use the "name, href" format as of right now. If it's not that critical, we can forgo this for a later issue.

mealthebear avatar Oct 11 '21 23:10 mealthebear

This is purely code style preferences: I noticed several files are inconsistent with ordering of keys in their breadcrumb links. (Ex.)

{ href: '/', name: 'Home' } vs { name: 'Home', href: '/' }  

It could be nice to change them to one standardized style, it seems like more files use the "name, href" format as of right now. If it's not that critical, we can forgo this for a later issue.

Good catch. I cleaned up the breadcrumb objects so href comes first

bruceplai avatar Oct 12 '21 01:10 bruceplai

Code Climate has analyzed commit 5914d400 and detected 3 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 3

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 80.9% (0.0% change).

View more on Code Climate.

codeclimate[bot] avatar Jun 12 '22 06:06 codeclimate[bot]