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

Release 7.3.1 #778

Merged
merged 5 commits into from
Jan 26, 2024
Merged

Release 7.3.1 #778

merged 5 commits into from
Jan 26, 2024

Conversation

JordanBalesBAO
Copy link
Collaborator

@JordanBalesBAO JordanBalesBAO commented Jan 25, 2024

Developer: Team

Reviewer:

  • Tests added/updated & passing.
  • Clean linting.
  • Related issue ticket in Jira/GitHub.
  • Documentation added/updated.
  • Code conforms to style guidelines.
  • Well commented.
  • Updates reflected in NEWS.md.
  • Build check passes.

JordanBalesBAO and others added 5 commits January 12, 2024 16:04
* Updated News and Description File

* testing if this is a local issue. Will comment out again if not

* Updated News and Description

* Updated NEWs

* Updated Description file, setup news.md

* Updated News.Md to relfect changes of PR

* Update Description and News

* Updated News.md

* fixed news.md entry error

* Updated Description & News

* Updated News

* Updated News

* Dp 1044 (#761)

* Regular Expression updated to catch {KP}. There was a period being searched.

* Added validation and removed NA filter.

* updated conditional logic to throw a warning

* reverted dropping NA's, log warning message when occurs.

* removed old warning message logic

* clean up

* fix bracket

* Correct fix is applied.

* Will only apply fix here, test will come in later

---------

Co-authored-by: Jordan <Jordan>

* DP-1180 (#762)

- Update to Year 2 tab for OVC to include 18-20 age band
- update default cell to A1 for all tabs

Co-authored-by: Jordan Bales <89547800+JordanBalesBAO@users.noreply.github.com>

* final news update for release

* Updated news and Description

* additional News

* Updated NEws and Description

* Updated News.MD

---------

Co-authored-by: Jordan <Jordan>
Co-authored-by: Christian Domaas <61550159+christian-domaas@users.noreply.github.com>
* Updated News and Description File

* testing if this is a local issue. Will comment out again if not

* Updated News and Description

* Updated NEWs

* Updated Description file, setup news.md

* Updated News.Md to relfect changes of PR

* Update Description and News

* Updated News.md

* fixed news.md entry error

* Updated Description & News

* Updated News

* Updated News

* Dp 1044 (#761)

* Regular Expression updated to catch {KP}. There was a period being searched.

* Added validation and removed NA filter.

* updated conditional logic to throw a warning

* reverted dropping NA's, log warning message when occurs.

* removed old warning message logic

* clean up

* fix bracket

* Correct fix is applied.

* Will only apply fix here, test will come in later

---------

Co-authored-by: Jordan <Jordan>

* DP-1180 (#762)

- Update to Year 2 tab for OVC to include 18-20 age band
- update default cell to A1 for all tabs

Co-authored-by: Jordan Bales <89547800+JordanBalesBAO@users.noreply.github.com>

* final news update for release

* Updated news and Description

* additional News

* Updated NEws and Description

* Updated News.MD

* commented code for future hurdle

* Added 2024 to have dedupe mechs

* removed comment

* Removed conditional logic as it relates to the year.

* Removed condition completely as dedupes should always be included in mechs.

* Updated comment to be representative.

---------

Co-authored-by: Jordan <Jordan>
Co-authored-by: Christian Domaas <61550159+christian-domaas@users.noreply.github.com>
@JordanBalesBAO JordanBalesBAO merged commit d420d95 into master Jan 26, 2024
1 check passed
@JordanBalesBAO JordanBalesBAO deleted the Release-7.3.1 branch January 26, 2024 12:54
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.

2 participants