Commit Graph

4285 Commits

Author SHA1 Message Date
Chris Hill-Scott
e7ed1c1cfe Line up linked message in viewport
We anchor link to the relevant message in a thread. Which is good, but
it leaves the messages hard against the top edge of the viewport. This
looks sloppy. So this commit:

- makes each message focusable
- shifts the focused message with CSS to sit away from the viewport
2017-06-10 12:03:10 +01:00
Chris Hill-Scott
f6d8e55579 Add two-way messaging view
> Once an inbound message has been received, there should be a way to
> see the other messages in the system from the same service to the same
> number. Both in and outbound. Nice inbox/whatsapp stylee view or some
> such. This way the context of the reply is understood.
>
> Initially will only see the outbound template, not the actual message,
> but we’re going to change this for the rest (soon), so that you can
> always see the full message for all outbound.
2017-06-10 12:03:10 +01:00
Pete Herlihy
bdca1f4395 54-55 for National Helpline (DfE) 2017-06-09 12:59:30 +01:00
venusbb
d5e3c00c0e Merge branch 'master' of https://github.com/alphagov/notifications-admin 2017-06-09 09:43:39 +01:00
venusbb
da90fd2a8b modified readme 2017-06-08 16:50:15 +01:00
Pete Herlihy
442b25143a Updated roadmap for June 2017 2017-06-08 15:25:37 +01:00
minglis
183c324f9a Merge pull request #1309 from alphagov/reinstate-new-rate-api
Reinstate new rate api
2017-06-08 14:07:33 +01:00
minglis
834c255195 Fixed typos 2017-06-08 13:55:13 +01:00
Chris Hill-Scott
d9a46a67bb Merge pull request #1306 from alphagov/roll-up-inbox
Roll up messages in inbox
2017-06-07 16:24:22 +01:00
Chris Hill-Scott
f150e2be12 Refactor on/off table fields into component
There’s a lot of repeated code in our Jinja templates for the settings
page.

We already have a `boolean_field` component, but it’s not used anywhere
(it was just lifted from Digital Marketplace).

So this commit changes the `boolean_field` component to be useful for
our on/off use case, and wires it into the settings page.
2017-06-07 15:07:20 +01:00
Chris Hill-Scott
363a3e1864 Add settings page for inbound SMS
Users might be interested in inbound SMS. And when it’s fully
available, they’ll probably be able to control whether it’s on/off for
their service.

Until they point, the only way of getting it is to ask us. So let’s make
an in-the-meantime page that directs them to ask us, from the place
where they’d be able to do it themselves.
2017-06-07 15:07:20 +01:00
Chris Hill-Scott
3a218c8c51 Fix page title on international page
It doesn’t match the `<H1>`
2017-06-07 15:06:40 +01:00
Martyn Inglis
784be721f4 Merge branch 'master' into reinstate-new-rate-api
Conflicts:
	app/main/views/dashboard.py
2017-06-07 14:44:27 +01:00
Martyn Inglis
d2ecd2121e Wired in the free limit from the API 2017-06-07 14:26:00 +01:00
Chris Hill-Scott
2b164733d8 Merge pull request #1307 from alphagov/quis-patch-1
Add handler for old feedback form
2017-06-07 12:36:03 +01:00
Chris Hill-Scott
4259fb02ba Add handler for old feedback form
Cool URIs don’t change 😎

https://www.w3.org/Provider/Style/URI

We still have links to `/feedback` in our emails. These will live in people’s inboxes forever.
2017-06-07 12:25:33 +01:00
Chris Hill-Scott
313f669690 Roll up messages in inbox
The inbox should work a bit like the one on your phone. You shouldn’t
see all the messages, but the latest one from each of your ‘contacts’
only.
2017-06-06 15:15:36 +01:00
Chris Hill-Scott
a49e1b4f75 Merge pull request #1305 from alphagov/fix-key-inbound
Fix timestamp of latest inbound text on dashboard
2017-06-06 12:47:31 +01:00
Chris Hill-Scott
67137f5570 Fix timestamp of latest inbound text on dashboard
Key was misnamed.
2017-06-06 12:44:59 +01:00
Chris Hill-Scott
9d0d74cce4 Merge pull request #1304 from alphagov/53-54
53-54
2017-06-06 12:23:58 +01:00
Pete Herlihy
bc494a1821 53-54 2017-06-06 12:01:15 +01:00
Chris Hill-Scott
772b5da85c Merge pull request #1303 from alphagov/data-gov-uk
Add DATA.GOV.UK logo as email branding option
2017-06-06 11:21:44 +01:00
Chris Hill-Scott
6a716ef162 Add DATA.GOV.UK logo as email branding option 2017-06-06 11:02:11 +01:00
Chris Hill-Scott
e929a6b245 Merge pull request #1289 from alphagov/inbound-dashboard
Show inbound messages on dashboard
2017-06-05 16:46:03 +01:00
Chris Hill-Scott
85105fd6cd Only show inbound stuff if service has permission
Only services that have inbound SMS turned on should be able to see the
dashboard and ‘Received messages’ page.

There’s probably a cleaner way (decorator) of doing this permissions
stuff, but I think it can wait until we ship this.
2017-06-05 15:52:30 +01:00
Chris Hill-Scott
95e613c3f9 Clarify that big numbers mean ‘sent’
Since we now have inbound and outbound messages we should be clear about
which the counts refer to.
2017-06-05 15:51:30 +01:00
Chris Hill-Scott
e373296bd9 Show inbound messages on the dashboard
This commit adds two things:

a section on the dashboard to show how many inbound messages the
service has received in the last 7 days, and how recently an inbound
message has been received
---

Doesn’t show the contents of any messages, just like how the rest of the
dashboard is an aggregation, never individual messages.

a page to show all the inbound messages the service has received in
the last 7 days
---

This shows the first line of the message. Eventually this will link
through to a ‘conversation’ page, where a service can see all the
messages it’s received from a given phone number.
2017-06-05 15:42:36 +01:00
Chris Hill-Scott
56354a0177 Merge pull request #1302 from alphagov/cqc
Add Care Quality Commission to email whitelist
2017-06-05 14:07:10 +01:00
Rebecca Law
e06f2a0d54 Merge pull request #1300 from alphagov/inbound-sms-permission
Service setting for inbound sms
2017-06-05 13:54:42 +01:00
Chris Hill-Scott
3d45454330 Add Care Quality Commission to email whitelist
CQC is an executive non-departmental public body, sponsored by the
Department of Health.

They have asked to be allowed to register for Notify using the
`cqc.org.uk` and `digital.cqc.org.uk` domains. We know that this really
is there domain because it’s linked to from here:

https://www.gov.uk/government/organisations/care-quality-commission
2017-06-05 13:51:53 +01:00
Rebecca Law
9d7510a52f Revert if statement refactor, if set_inbound_sms is False it still resolves to True, because it is not a boolean but a string. 2017-06-05 13:46:02 +01:00
Rebecca Law
da77737c16 Refactor if statement 2017-06-05 12:13:40 +01:00
Rebecca Law
f40e81bc3a Added a test to turn off the inbound permission. 2017-06-05 12:01:39 +01:00
Chris Hill-Scott
6e6f3ad794 Merge pull request #1301 from alphagov/fix-weird-meta-on-activity-page
Don’t link to one-off reports from activity
2017-06-05 11:15:21 +01:00
Chris Hill-Scott
e4bf0c3e7d Don’t link to one-off reports from activity
There’s no need to see the ‘Report’ page for a one-off message – pretty
much all the info about the notification is there on the page. And
‘Sent from _Report_’ looks weird.

We should display it more like an API call instead.
2017-06-02 17:03:41 +01:00
Rebecca Law
1960a252ea Fix merge conflict and typo in html 2017-06-02 16:37:45 +01:00
Rebecca Law
fe24501075 Add a platform admin button to service-settings page to turn the inbound_sms messaging on and off.
If clicked you will be prompted to enter a sms sender number, when setting the permission on or off.
Team members will always be able to see the number, but will only be able to change it if the inbound_sms permission is off.
2017-06-02 16:33:31 +01:00
Chris Hill-Scott
4a251203df Merge pull request #1286 from alphagov/no-sender-fallback
Remove fallback for empty SMS sender
2017-06-02 12:16:52 +01:00
Chris Hill-Scott
fc9e8a6ee1 Merge pull request #1299 from alphagov/52-53
52-53 for Immigration Asylum Chamber Tribunal
2017-06-02 12:16:45 +01:00
Pete Herlihy
b73a306718 52-53 for Immigration Asylum Chamber Tribunal 2017-06-02 11:26:22 +01:00
minglis
e478d2f820 Merge pull request #1298 from alphagov/revert-dashboard-changes
Revert "Merge pull request #1288 from alphagov/add-new-rate-logic"
2017-06-02 10:50:58 +01:00
Martyn Inglis
0dbd3fa10d Revert "Merge pull request #1288 from alphagov/add-new-rate-logic"
This reverts commit adbff63d1f, reversing
changes made to 80f0b4b2a2.
2017-06-02 10:41:42 +01:00
minglis
adbff63d1f Merge pull request #1288 from alphagov/add-new-rate-logic
Wire in the new API method that calculates the total cost and total billable units
2017-06-02 09:52:19 +01:00
Martyn Inglis
790aa250e4 Merge branch 'add-new-rate-logic' of github.com:alphagov/notifications-admin into add-new-rate-logic 2017-06-01 17:05:07 +01:00
Martyn Inglis
e236cc4cbe Fixed typo and moved the sms free limit into the config. 2017-06-01 17:04:41 +01:00
minglis
8f70dcf549 Fix typo 2017-06-01 16:55:49 +01:00
minglis
74e79e175b Fix typo 2017-06-01 16:53:36 +01:00
Chris Hill-Scott
80f0b4b2a2 Merge pull request #1297 from alphagov/search-outbound
Add search box to filter notifications by recipient
2017-06-01 16:09:20 +01:00
Chris Hill-Scott
20f1d73f76 Merge pull request #1293 from alphagov/one-off
Make ‘send a one-off message’ a thing
2017-06-01 14:46:07 +01:00
Chris Hill-Scott
f12e0fde39 Ensure the tour sidebar gets shown
I accidentally broke it by removing a parameter. This commit reinstates
that parameter and adds some tests to make sure it doesn’t happen again.
2017-06-01 14:27:55 +01:00