Commit Graph

9296 Commits

Author SHA1 Message Date
karlchillmaid
10e0c39001 Add reference for letter_spec_guidance 2020-02-13 15:53:03 +00:00
karlchillmaid
8a0a9666d7 Update links 2020-02-13 15:50:22 +00:00
karlchillmaid
fadf29c9b0 Fix API validation error summary links 2020-02-13 15:08:40 +00:00
karlchillmaid
90462d7c71 Update link to Letter specification guidance 2020-02-13 14:58:53 +00:00
karlchillmaid
29c5412a8d Add filetypes to links 2020-02-13 13:06:54 +00:00
karlchillmaid
ce553317c6 Remove file types
Remove file types from links while I check the GOV.UK style guide
2020-02-12 15:01:17 +00:00
karlchillmaid
e27b58524f Add anchor to link 2020-02-12 14:24:49 +00:00
karlchillmaid
15aa876f4c Add anchor 2020-02-12 14:23:53 +00:00
karlchillmaid
600cc49345 Add link to guidance 2020-02-12 14:22:25 +00:00
karlchillmaid
8fb9668775 Add letter specification guidance
Add letter specification information to the guidance
2020-02-12 14:13:50 +00:00
David McDonald
e78ea10b49 Merge pull request #3292 from alphagov/pyup-scheduled-update-2020-02-10
Scheduled weekly dependency update for week 06
2020-02-12 13:40:54 +00:00
pyup-bot
2597a70d5f Update flask-wtf from 0.14.2 to 0.14.3 2020-02-12 13:33:25 +00:00
karlchillmaid
38fd842d33 Merge pull request #3213 from alphagov/add-guidance-content
Add first iteration of guidance content, including updating navigation menus and footer.
2020-02-12 11:43:07 +00:00
karlchillmaid
98ccce5d90 Make click-throughs content clearer 2020-02-12 11:33:10 +00:00
karlchillmaid
d24d772cfa Reorder introduction content 2020-02-12 11:25:58 +00:00
karlchillmaid
0949fc6480 Update app/templates/views/guidance/edit-and-format-messages.html 2020-02-12 11:17:26 +00:00
karlchillmaid
4d7a41d55d Merge pull request #3289 from alphagov/change-callout-to-inset-text
Change callout to inset text
2020-02-12 11:17:07 +00:00
Leo Hemsted
b0ee3f22d9 Merge pull request #3283 from alphagov/pyup-scheduled-update-2020-02-03
Scheduled weekly dependency update for week 05
2020-02-11 16:30:04 +00:00
Tom Byers
1d0c2070db Merge pull request #3291 from alphagov/fix-pill-heights
Make pill items match their heights
2020-02-11 14:28:14 +00:00
Chris Hill-Scott
6b703614b9 Merge pull request #3280 from alphagov/swap-nav-order-org-users
Swap order of usage and team members in navigation when organisation users are looking at a service
2020-02-11 11:21:14 +00:00
Chris Hill-Scott
8a4bf1d07e Make test more robust
Now tests:
- that the user doesn’t have the usage menu item when they’re not a
  member of the organisation (i.e. the counterfactual)
- that the user still gets a 403 when they try to view the usage page if
  they’re not a member of the organisation
2020-02-11 11:11:18 +00:00
Tom Byers
4433db346b Center pill content using more flexbox
Turns each pill item (already full height) into a
flex container with its content as a column. Uses
this to align the content (the pill label) to the
middle of the column.
2020-02-07 17:42:53 +00:00
Tom Byers
c8d9625518 Merge pull request #3164 from alphagov/update-static-error-pages
Update static error pages
2020-02-07 12:49:47 +00:00
Tom Byers
76240d5d55 Fix issues from pull request review
Fixes for the following:
- `<p>` tags need `govuk-body` class
  - https://github.com/alphagov/notifications-admin/pull/3164#discussion_r375921631
  - https://github.com/alphagov/notifications-admin/pull/3164#discussion_r375924387
- unused JS file
  - https://github.com/alphagov/notifications-admin/pull/3164#discussion_r375949386
- link to GDS has incorrect href
  - https://github.com/alphagov/notifications-admin/pull/3164#discussion_r375966464
  - https://github.com/alphagov/notifications-admin/pull/3164#discussion_r375966555
2020-02-07 11:21:11 +00:00
Tom Byers
a5ea785474 Make pill items match their heights 2020-02-06 16:38:24 +00:00
karlchillmaid
f7e1e35dcb Remove commented-out content
Co-Authored-By: Tom Byers <tombaromba@gmail.com>
2020-02-06 16:06:00 +00:00
karlchillmaid
77590bb9d3 Remove commented-out content
Co-Authored-By: Tom Byers <tombaromba@gmail.com>
2020-02-06 16:05:32 +00:00
karlchillmaid
d646238a01 Remove commented-out content
Co-Authored-By: Tom Byers <tombaromba@gmail.com>
2020-02-06 16:03:48 +00:00
karlchillmaid
ccb783c43b Update app/templates/views/guidance/index.html
Co-Authored-By: Tom Byers <tombaromba@gmail.com>
2020-02-06 16:02:41 +00:00
karlchillmaid
a5c77a6fc3 Remove commented-out content 2020-02-06 16:01:23 +00:00
karlchillmaid
74874ee78d Remove commented-out content 2020-02-06 15:59:07 +00:00
karlchillmaid
79373909bd Add 'add' 2020-02-06 12:54:27 +00:00
karlchillmaid
5775ccb5b1 Change callout to inset text 2020-02-06 12:30:22 +00:00
Rebecca Law
94c9886a58 Merge pull request #3288 from alphagov/delete-template-dialog
Delete template dialog
2020-02-06 10:58:43 +00:00
Rebecca Law
b242467d77 Change the content for the delete template confirmation dialog.
As per https://www.pivotaltracker.com/story/show/170796514 we want to make the delete template confirmation dialog box more consistent and clear.
The API has been updated with a new endpoint that only returns the last-used date, this date is more accurate since it goes to the ft_notification_status table, if the notification table is empty.
2020-02-06 10:16:00 +00:00
Rebecca Law
60dd6bca44 Merge pull request #3287 from alphagov/check-current-user-is_authenticated
Reorder the methods called in sign out
2020-02-05 16:29:31 +00:00
Rebecca Law
039628cff7 Reorder the methods called in sign out
We found another scenario where signing out of the db can cause a 500.
If the user archives their trial mode service, current_service.active = false, then signs out, the current user was being signed out client side first, meaning current_user is now an Anonymous user, next the call to the API is made to log out user on db, all calls to NotifyApiClient `check_inactive_service`, which is only authorised if user is platform_admin, but an AnonymousUser does not have that attribute, so a 500 is raise.
Seemed a bit cleaner to change the User.signout method to rather than the `check_inactive_service` method for current_user.is_authenticated.
2020-02-05 15:58:15 +00:00
Matt Hobbs
a4a6ad04f7 Merge pull request #3279 from alphagov/image-optimisation
Image optimisation
2020-02-05 10:46:42 +00:00
Tom Byers
2bb651865a Remove unused images
As per @quis's comment:

https://github.com/alphagov/notifications-admin/pull/3279#issuecomment-581902125
2020-02-05 10:37:57 +00:00
Leo Hemsted
636a702d19 Merge pull request #3286 from alphagov/remove-api-keys
cleanup
2020-02-05 10:11:47 +00:00
karlchillmaid
c7983c2396 Update app/templates/views/get-started.html
Fix b tag

Co-Authored-By: Tom Byers <tombaromba@gmail.com>
2020-02-05 09:24:59 +00:00
Leo Hemsted
4eeddb40f8 bump reqs 2020-02-04 17:16:58 +00:00
Pea M. Tyczynska
b5723a1aef Merge pull request #3285 from alphagov/uploads-fix-letter-link
Fix bug where job url type was constructed for letter on uploads page
2020-02-04 16:26:49 +00:00
Tom Byers
fbff4beef5 Update app/assets/error_pages/5xx.html
Remove unneeded GOV.UK suffix from page title on error page.

Co-Authored-By: karlchillmaid <karl.chillmaid@digital.cabinet-office.gov.uk>
2020-02-04 16:07:33 +00:00
Tom Byers
0d80a4cc8b Update paas-failwhale/index.html
Remove unneeded GOV.UK suffix from page title.

Co-Authored-By: karlchillmaid <karl.chillmaid@digital.cabinet-office.gov.uk>
2020-02-04 16:07:17 +00:00
Rebecca Law
143564c7f0 Update message seen when deleting a template. 2020-02-04 16:00:51 +00:00
Pea Tyczynska
9ebd18ea85 Add upload_type property to Job class 2020-02-04 15:36:55 +00:00
Leo Hemsted
5b868ff268 remove api keys from codebase
these are just used for tests so don't need to look like real api keys.
2020-02-04 14:44:19 +00:00
Rebecca Law
fd59491d19 Merge pull request #3284 from alphagov/fix-sign-out-bug
Check if user is authenticated before signing out
2020-02-04 14:37:08 +00:00
Pea Tyczynska
46292949cd Fix bug where job url type was constructed for letter on uploads page 2020-02-04 14:19:07 +00:00