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

update jac Kit for Qt DL button #1665

Merged
merged 3 commits into from
Jul 27, 2022

Conversation

tomlovesgithub
Copy link
Contributor

What's included?

JAC-kit update amends the styling which was causing an issue which made the buttons un-clickable.
This issue was raised specifically around QT response downloads.

Who should test?

✅ Product owner
✅ Developers
✅ UTG

How to test?

  • Navigate to the responses view of a QT.
  • Check download button is clickable.

Risk - how likely is this to impact other areas?

🟢 No risk - this is a self-contained piece of work


PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

Copy link
Member

@warrensearle warrensearle left a comment

Choose a reason for hiding this comment

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

Code change looks fine to me 👍

Are there any other changes/fixes in JAC Kit we should be aware that we are introducing?

Copy link
Contributor

@HalcyonJAC HalcyonJAC left a comment

Choose a reason for hiding this comment

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

Code look great to me!

@tomlovesgithub tomlovesgithub force-pushed the update-jac-kit-qt-response-download branch from 5e8e97a to 84e5fb5 Compare July 15, 2022 16:18
@tomlovesgithub tomlovesgithub temporarily deployed to develop July 15, 2022 16:23 Inactive
@github-actions
Copy link

Visit the preview URL for this PR (updated for commit b1154d8):

https://jac-admin-develop--pr1665-update-jac-kit-qt-re-1kblkj8d.web.app

(expires Sun, 14 Aug 2022 16:27:43 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@tomlovesgithub tomlovesgithub merged commit 7f4e88f into main Jul 27, 2022
@tomlovesgithub tomlovesgithub deleted the update-jac-kit-qt-response-download branch July 27, 2022 08:44
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.

4 participants