17101 Commits
Author SHA1 Message Date
Pete Herlihy e8716d643d Updated contact us links on request to go live page to point at new feedback page 2016-04-20 11:58:03 +01:00
NIcholas Staples 4f71c40735 Merge pull request #466 from alphagov/desktop_feedback
Feedback page added. All tests working.
2016-04-20 11:17:55 +01:00
Chris Hill-Scott 8d6eb0985b Merge pull request #471 from alphagov/craig-david
Show the last 7 days of statistics on the dashboard
2016-04-20 11:13:51 +01:00
NIcholas Staples 49e0a98c2c Merge pull request #473 from alphagov/doc_page_fix
Add encoding to the documentation file.
2016-04-20 11:05:59 +01:00
Nicholas Staples bd0a572faf Add encoding to the documentation file. 2016-04-20 10:58:23 +01:00
Chris Hill-Scott 3391b22e73 Merge pull request #472 from alphagov/quis-patch-1
Let everyone see the API info page
2016-04-20 10:36:05 +01:00
Nicholas Staples da536bbd2e Feedback page working with all tests passing.
Updated to include team id.

Give Feedback -> Give feedback
2016-04-20 10:17:09 +01:00
Chris Hill-Scott be7c7cebbc Limit big numbers to the last 7 days
Implements and depends on:
- https://github.com/alphagov/notifications-api/pull/241
2016-04-19 17:10:39 +01:00
Chris Hill-Scott 0b383e2fae Let everyone see the API info page
If it’s not for them it’s OK, they’ll be scared away.
2016-04-19 16:36:35 +01:00
Chris Hill-Scott 5bf0d8fe70 Show all statistics fetched on the dashboard
We want to align all our stats to be for the last 7 days.

This means summing up the stats response from the API to make the Big
Number. Previously the big number was counting sent notifications as
successful. This commit changes it to only look at delivered
notifications.

Right now, the API doesn’t have a way of filtering to only show the last
7 days. So for the moment the dashboard will show statistics for all
time.

The upshot of this is that we can link from the dashboard to the
activity page when there are failures.
2016-04-19 15:57:53 +01:00
Chris Hill-Scott 9b75f7ef8a Merge pull request #467 from alphagov/tour-footer-link
Put a link to the tour in the footer
2016-04-19 15:55:47 +01:00
Chris Hill-Scott 352750dcf6 Merge pull request #470 from alphagov/better-better-send-page
Make the flow of using templates better
2016-04-19 15:54:47 +01:00
NIcholas Staples cd1351bd06 Merge pull request #468 from alphagov/download_csv_all
Functionality added to download all on activity page. All tests passing.
2016-04-19 15:46:41 +01:00
Chris Hill-Scott 8d3fa79180 Reorder navigation to put emails before SMS
Elsewhere (eg the dashboard, the activity page) we have email first.
Emails are also anticipated to be slightly more popular in terms of the
number of services sending them.

Therefore it makes sense to have emails first in the main navigation.
2016-04-19 15:31:19 +01:00
Chris Hill-Scott 27ad1532e4 Make the flow of using templates better
For users who:
- want to send messages from a template
- want to edit templates

For developers:
- who need to get the ID of a template

This commit mainly cleans up the choose template page so there are less
options, and the options that are there are less wordy.

This means:
- moving ‘send yourself a test’ onto the send messages page, and making
  it button
- stripping a lot of stuff out of the ‘send from API’ page, so it’s more
  obvious what the template ID is
2016-04-19 15:31:19 +01:00
NIcholas Staples ddec619913 Merge pull request #469 from alphagov/fix_doc_link
Doc file is loaded using current directory of the flask view.
2016-04-19 13:40:23 +01:00
Nicholas Staples e8dc03f121 Doc file is loaded using current directory of the flask view. 2016-04-19 13:33:28 +01:00
Nicholas Staples f3210ea992 Functionality added to download all on activity page. All tests passing. 2016-04-19 11:45:36 +01:00
Chris Hill-Scott 2a080835aa Put a link to the tour in the footer
So you can:
- see it again
- see it if you don’t have an account yet
2016-04-19 11:07:48 +01:00
Chris Hill-Scott b75ab7c3df Make new service current service on first use
When you add a new service, it’s probably the one you want to do stuff
with.

When you get invited, the service you’ve been invited to is probably the
one you want to use.

This commit adds the ID of the new service or service you’ve been
invited to to the session.
2016-04-19 11:04:00 +01:00
Chris Hill-Scott 37f341e757 Make tour work for non-logged-in users
There’s no content in the tour that’s specific to a service. And since
we can now take a pretty good guess at what service you last used, or
which service we should send you to if you only have one service,
there’s no need to make the URLs for the tour service-specific.

This also means that you don’t need to be logged in to see the tour
pages, and we have no good reason to only restrict these pages to users
with accounts.

https://www.pivotaltracker.com/story/show/116960703
2016-04-19 10:38:16 +01:00
Chris Hill-Scott c2b96be247 Merge pull request #464 from alphagov/correct-innacurate-csv-error
Fix innacurate error message on CSV check
2016-04-18 16:52:52 +01:00
Rebecca Law 6c6dc51ea3 Revert changes to aws deployment location for master builds on preview environment 2016-04-18 14:13:38 +01:00
Chris Hill-Scott a0e3a648f3 Fix innacurate error message on CSV check
We only know if a row contains missing cell(s), not how many cells are
missing data.

This commit changes the error message to be more accurate.
2016-04-18 11:47:53 +01:00
Chris Hill-Scott fb111b2968 Merge pull request #463 from alphagov/placeholder-hint-sms
Make placeholder hint work for SMS messages
2016-04-18 10:30:37 +01:00
Chris Hill-Scott a63c50b0d5 Merge pull request #459 from alphagov/wrap-long-words-sms-template
Wrap long words in SMS template previews
2016-04-18 10:30:26 +01:00
Chris Hill-Scott a08764db71 Merge pull request #458 from alphagov/reverse-filters-activity
Reverse the order of the filters on activity page
2016-04-18 10:30:18 +01:00
Chris Hill-Scott cfeb7514c3 Make placeholder hint work for SMS messages
The changes introduced in 54e42a2021
didn’t get applied to the add/edit SMS template page. This commit fixes
that.
2016-04-15 17:16:18 +01:00
Rebecca Law b242106f11 Merge pull request #460 from alphagov/aws-fixup
Update deployment files for aws
2016-04-15 16:42:32 +01:00
Rebecca Law 0b777b76a6 Update deployment files for aws 2016-04-15 16:31:31 +01:00
Chris Hill-Scott 5dfbd267ce Wrap long words in SMS template previews
If you put, for example, a URL in an SMS template it can be very long.
This can cause it to overflow its container. This commit forces it to
wrap instead.

Related: https://github.com/alphagov/notifications-admin/pull/439
2016-04-15 15:11:44 +01:00
Rebecca Law f606422c1a Fix code deploy names 2016-04-15 14:59:02 +01:00
Chris Hill-Scott 7ddaf56aa9 Reverse the order of the filters on activity page
So that they match the order of the page title, eg ‘Failed text
messages’
2016-04-15 14:05:00 +01:00
Chris Hill-Scott 9c0d7c17cf Merge pull request #456 from alphagov/docs-in-app
Put the new documentation in the app
2016-04-15 12:27:51 +01:00
Chris Hill-Scott 0a60ae24bf Merge pull request #451 from alphagov/show-subject-choose-template
Show email subject on choose template page
2016-04-15 12:27:35 +01:00
Chris Hill-Scott e89063e285 Move the image so the app can serve it 2016-04-15 12:15:30 +01:00
Chris Hill-Scott f7c15dfbaf Add some basic typography, tweak the Markdown
This commit adds some basic type sizes, weights and spacing for HTML
elements scoped inside a `.documentation` container.

It also tweaks some of the Markdown so that it renders the same as it
does on Github.
2016-04-15 12:15:30 +01:00
Chris Hill-Scott 57ad64fae6 Get the docs rendering in the app
This commit replaces the old _API Documentation_ page with the Markdown
version that Catherine has been working on.

I’ve checked that there’s nothing obviously wrong or placeholder-y still
in there, so I think we’re good to go.
2016-04-15 12:14:51 +01:00
Rebecca Law 3c87af44ee Merge pull request #454 from alphagov/simplify-jwt
Simplify JWT
2016-04-15 12:08:41 +01:00
Rebecca Law 848db38573 Added missing import 2016-04-15 11:53:39 +01:00
NIcholas Staples b85c6c3b1a Merge pull request #455 from alphagov/add_created_by_to_api_calls
Add created_by to all appropriate requests.
2016-04-15 11:14:05 +01:00
Nicholas Staples 2085792742 Add created_by to all appropriate requests. 2016-04-15 11:08:19 +01:00
Rebecca Law 1fe5ae0371 This pull request removes the need for the req and pay in the claims JWT.
Use the new version of the notifications-python-client. This version no longer adds the req and pay to the claims of the jwt.

The change is backward compatible so an older client that sends a JWT with the extra claims will pass authentication.
Once all the clients have been updated to not include the extra claims some updates to exclude them from the method signatures will happen as well.

The documentation has been updated to reflect this change.

https://www.pivotaltracker.com/story/show/116971293
2016-04-15 11:04:35 +01:00
catherineheywood 99d5a3afc7 Doc changes
Add links to PHP and Java libraries
Define sent_at
Indicate how to access template IDs
2016-04-14 16:40:30 +01:00
Chris Hill-Scott c71249a7ea Show email subject on choose template page
Removed this before as part of truncating email previews in
https://github.com/alphagov/notifications-admin/commit/3a5b76ce2a672ba5d42b4013a0274f64742eb5d7#diff-b5f54dc364655c298fd119e3bc148cc6R45

But actually trying to use the app it’s a weird inconsistency to not
have the subject show everywhere.

So this commit reinstates it.
2016-04-14 15:43:19 +01:00
NIcholas Staples 51673e26f4 Merge pull request #450 from alphagov/make_service_live
Make service live and revert working, all tests passing.
2016-04-14 15:43:09 +01:00
Nicholas Staples 6d9de1a500 Make service live and revert working, all tests passing. 2016-04-14 15:38:34 +01:00
Chris Hill-Scott 12053a4532 Merge pull request #449 from alphagov/detect-placeholders-in-email-subject
Detect placeholders in email subject
2016-04-14 15:25:54 +01:00
Chris Hill-Scott a6279f4114 Move placeholder hint to align with subject
If the hint aligns with the message body then it looks like you can only
add placeholders to the body. This commit aligns it with the  subject
instead, and since it overlaps both it looks like you can add
placeholders to both (hopefully).
2016-04-14 14:56:59 +01:00
Chris Hill-Scott 54e42a2021 Make the hint work for both subject and body
This commit makes the placeholder hint not be dependent on their only
being one textbox with placeholders on a page.

To do this it needs to be its own module, because the two textboxes are
each self-contained instances of the same module.

The way I’ve gone about this is to make the placeholder hint accept a
list of textboxes it should look in when deciding what message to
display to the user.

Most of the code is extracted from the existing placeholder highlighting
module.
2016-04-14 14:54:53 +01:00