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

Set Login Year to 2023 #1899

Merged
merged 6 commits into from
Aug 1, 2023
Merged

Set Login Year to 2023 #1899

merged 6 commits into from
Aug 1, 2023

Conversation

ailZhou
Copy link
Collaborator

@ailZhou ailZhou commented Jul 27, 2023

Description

We're finally doing it, we're making the login default reporting year 2023 in QMR.

Also, cypress test have been updated to select year 2021 so that it doesn't break everything.

Related ticket(s)

MDCT-2764


How to test

  • Sign into QMR
  • Look at the Reporting Year, it's now 2023

We will never have to make this mistake again.

Important updates

N/A


Author checklist

  • I have performed a self-review of my code
  • I have added thorough tests, if necessary
  • I have updated relevant documentation, if necessary

convert to a different template: test → val | val → prod

@ailZhou ailZhou marked this pull request as ready for review July 27, 2023 20:51
@ailZhou ailZhou added ready for review Ready for all the reviews! product review Waiting for product review labels Jul 27, 2023
britt-mo
britt-mo previously approved these changes Jul 27, 2023
gmrabian
gmrabian previously approved these changes Jul 28, 2023
Copy link
Contributor

@gmrabian gmrabian left a comment

Choose a reason for hiding this comment

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

🙌
one line change for the code and 142 lines for cypress 😭

karla-vm
karla-vm previously approved these changes Jul 28, 2023
Copy link
Contributor

@karla-vm karla-vm left a comment

Choose a reason for hiding this comment

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

FINALLY!

@ailZhou ailZhou dismissed stale reviews from karla-vm, gmrabian, and britt-mo via 3d3935f July 28, 2023 15:28
services/ui-src/src/views/AdminHome/index.tsx Outdated Show resolved Hide resolved
services/ui-src/src/views/Home/index.tsx Outdated Show resolved Hide resolved
@codeclimate
Copy link

codeclimate bot commented Jul 31, 2023

Code Climate has analyzed commit f5dac75 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 50.0% (90% is the threshold).

This pull request will bring the total coverage in the repository to 72.7% (0.0% change).

View more on Code Climate.

@ailZhou ailZhou removed the product review Waiting for product review label Jul 31, 2023
Copy link
Contributor

@BearHanded BearHanded left a comment

Choose a reason for hiding this comment

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

Looks good!

@BearHanded BearHanded merged commit d95a37f into master Aug 1, 2023
166 checks passed
@BearHanded BearHanded deleted the mdct-2764-login-year branch August 1, 2023 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review Ready for all the reviews!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants