Skip to content
This repository has been archived by the owner on Jun 13, 2020. It is now read-only.

Updates Readme #32

Open
wants to merge 52 commits into
base: master
Choose a base branch
from
Open

Updates Readme #32

wants to merge 52 commits into from

Conversation

anselmbradford
Copy link
Member

Changes

  • Updates test URL in readme.

rosskarchner and others added 30 commits November 18, 2016 15:40
This reverts commit 2554d1e.
update data set link to bypass new socrata "primer" page
Remove reference to google analytics
Update content on complaints landing page
Modifying api endpoints in js and html
As part of the first release of Mosaic, the CCDB landing page needs
several updates:

- Remove the self-updating complaint number
- Remove the complaint randomizer
- Remove the data downloader (and replace it with a few static links)
- Update some text in the about the data section
- Add a note about recent CCDB updates
The section on the recent CCDB changes needed to be more noticable, so
we've moved it into a well above the "About the data" section.
We had a few small content updates to make after the CCDB landing page
went through clearance.
Add links to CCDB crosswalk PDFs
niqjohnson and others added 22 commits May 2, 2017 15:33
Fix tests for running with cfgov-refresh
Fix a few typos on the CCDB page
Fixes missing whitespace in h3
Update readme and changelog
Remove feature flag for landing page
Closing quote to resolve INSTALLED_APPS syntax issues.
allow technical issues banner to be enabled on-demand
It created a dependency on Socrata, and the results were no longer used
build wheels and attach them to new releases
Copy link
Member

@willbarton willbarton left a comment

Choose a reason for hiding this comment

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

Actually, within the context of the readme, this URL is correct.

It's only when running as part of cfgov-refresh that the URL is different.

It might be worth adding a new section to the readme, but as the readme is written for running this app standalone, the new URL won't work.

Relevant distinctions:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants