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: 2023-09-05 #643

Merged
merged 18 commits into from
Sep 6, 2023
Merged

Release: 2023-09-05 #643

merged 18 commits into from
Sep 6, 2023

Conversation

ludtkemorgan
Copy link
Collaborator

Pulls the following changes over from core

The following changes are also from Core, but were missed a few months ago so were never carried over

dependabot bot and others added 18 commits September 5, 2023 18:08
* fix: add sorting by status

* fix: change default sort to status

* fix: default sorting by status

* fix: sort by application when the same status

* fix: orderBy test using default value

* fix: trigger backend sorting on initial load

* fix: add unsort icon to indicate that column might be sorted

* test: rollback test changes

* fix: use enums to order by status

* chore: upgrade uic
* feat: update label for monthly minimum income

* fix: change string to Minimum Monthly Income
* fix: add backend url validation for url fields

* fix: validate only if externalUrl should be URL

* fix: use form values for application types on failed form submit

* fix: hide url field when no digital application

* fix: set proper default value to common digital application choice

* fix: add hasHttps decorator for second error variant
@netlify
Copy link

netlify bot commented Sep 5, 2023

Deploy Preview for housing-sanjoseca-gov ready!

Name Link
🔨 Latest commit a20eef2
🔍 Latest deploy log https://app.netlify.com/sites/housing-sanjoseca-gov/deploys/64f7b801579ae80009ac73a4
😎 Deploy Preview https://deploy-preview-643--housing-sanjoseca-gov.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@YazeedLoonat YazeedLoonat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ludtkemorgan ludtkemorgan merged commit b66af87 into main Sep 6, 2023
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

5 participants