Skip to content
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

Update index.html #4935

Merged

Conversation

sallybracegirdle
Copy link
Contributor

@sallybracegirdle sallybracegirdle commented Apr 24, 2024

Updated copy to link to the new Open311 API information page, and removed links to broken Open311 website. [skip changelog]

Copy link
Member

@dracos dracos left a comment

Choose a reason for hiding this comment

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

As well as the missing link - this page is shown at /open311 on all installations of FixMyStreet, not just .com. So the language used either needs to make sense for all installations, or .com specific bits need to be put behind a [% IF c.cobrand.moniker == 'fixmystreet' %] section (as you can see in part of the removed text here).

templates/web/base/open311/index.html Outdated Show resolved Hide resolved
@sallybracegirdle
Copy link
Contributor Author

@dracos Ok, I have edited this and it hopefully works better now?

@dracos dracos changed the base branch from master to sallybracegirdle-new-page May 3, 2024 08:37
@dracos dracos merged commit 8585f0b into sallybracegirdle-new-page May 3, 2024
0 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants