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

Sprint 30-05-2017 14-06-2017 #215

Merged
merged 76 commits into from
Jun 20, 2017
Merged

Sprint 30-05-2017 14-06-2017 #215

merged 76 commits into from
Jun 20, 2017

Conversation

serge-gaia
Copy link
Contributor

@serge-gaia serge-gaia commented Jun 14, 2017

Warning! This MR requires migrations.

This MR includes the following kanboad cards (https://kanboard.dpaw.wa.gov.au/?controller=BoardViewController&action=show&project_id=24):
*3407
*2728
*3226
*2897
*3133
*2759
*3132
*3221
*3222
*3220
*3268
*3411
*3287
*3223
*3191
*3224


This change is Reviewable

tony-gaia and others added 30 commits May 30, 2017 20:40
Doing that fixed two bugs
Added some permission mixins specific to returns
Fixed formatting of application pdf, including rendering of different
field type values.

Data under conditions will render indented from conditional field.

Added link for PDF in officer / assessor tables.

Added unit tests.
…esting

Added tests for returns views permissions.
Added default period for licence type.

Initial issue licence form's end date field is the start date plus the
default period, or not set if the default period not set.

Added unit tests for default period.
Added view for deleting application session (and temp application).

Added javascript logic to call above view if user navigates away from
select licence type page (other than selecting licence).

Dashboard tables no longer delete temp applications.
Similar to application request amendments.
Date of birth needs to be no less than 1900 for strformat to work.
New navigation.js to delete session if unloaded.
…nsistently.

Changed breadcrumbs not to show Home or Select Licence type.

Added non-link bread crumb to say New/Edit/Amend/Reissue Application.
For application model, when calling the 'reference' property, check that
there is a lodgement and sequence number, otherwise return empty string.

For pdf PageTemplate, pass in list of frames, rather than just frame
object. Updated pdf generators in main app pdf module to also do this.

Fixed unit test descriptions.
serge-gaia and others added 26 commits June 8, 2017 16:00
…nst_exceptions

Consolidate dashboard against exceptions
Created data migration
Removed the BooleanField is_licence_amendment replaced it by a property.
Assessor display type
Updated migration to use new_licence rather than 'new licence'

Removed template / javascript that referenced is_licence_amendment

Changed success message to warning when saving conditions
Removed the proxy column in officer's application dashboard
… application during entry and trying to enter multiple applications simultaneously.
Created a base AuthMixin
Revisit all the mixins to use the new baseclass
Fixed an error in the TemplateEmailBase who add a wrong template name.
@coveralls
Copy link

coveralls commented Jun 14, 2017

Coverage Status

Coverage increased (+5.5%) to 55.646% when pulling 6fc2779 on gaiaresources:master into a238e40 on parksandwildlife:master.

@wilsonc86
Copy link
Contributor

:lgtm:


Reviewed 76 of 76 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@dbca-asi dbca-asi merged commit a26cc78 into dbca-wa:master Jun 20, 2017
dbca-dragon pushed a commit to dbca-dragon/ledger that referenced this pull request May 27, 2019
WildlifeCompliance: licence api add filter for licence+activity lookup
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

5 participants