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 v0.86.0 #1369

Merged
merged 5 commits into from
May 13, 2024
Merged

release v0.86.0 #1369

merged 5 commits into from
May 13, 2024

Conversation

dcshzj
Copy link
Contributor

@dcshzj dcshzj commented May 13, 2024

New

  • fix(sl): fix error code #1366
  • feat: optimise local diff #1334
  • chore(sitelaunch): better error message #1358
  • backport v0.85.0 #1363

Tests

fix(sl): fix error code #1366

tldr; below we are testing for

  1. domains that have caa + aaaa record (google.com)
  2. domains that dont resolve to a host name (dontexist.sg)
  3. domains that do resolve to a host name but dont have an aaaa/caa record (isomer.gov.sg)

Locally enter these values into support/routes/v2/formsg/index.ts

formsgSiteLaunchRouter
  .digAAAADomainRecords({
    primaryDomainSource: "isomer.gov.sg",
  } as SiteLaunchResult)
  .then(console.log)
  .catch(console.error)

formsgSiteLaunchRouter
  .digCAADomainRecords({
    primaryDomainSource: "isomer.gov.sg",
  } as SiteLaunchResult)
  .then(console.log)
  .catch(console.error)
formsgSiteLaunchRouter
  .digAAAADomainRecords({
    primaryDomainSource: "dontexist.sg",
  } as SiteLaunchResult)
  .then(console.log)
  .catch(console.error)

formsgSiteLaunchRouter
  .digCAADomainRecords({
    primaryDomainSource: "dontexist.sg",
  } as SiteLaunchResult)
  .then(console.log)
  .catch(console.error)
formsgSiteLaunchRouter
  .digAAAADomainRecords({
    primaryDomainSource: "google.com",
  } as SiteLaunchResult)
  .then(console.log)
  .catch(console.error)

formsgSiteLaunchRouter
  .digCAADomainRecords({
    primaryDomainSource: "google.com",
  } as SiteLaunchResult)
  .then(console.log)
  .catch(console.error)

assert that the console log outputs

support-1   | [08:59:33.272] INFO (56): Domain does not have any AAAA records.
support-1   |     module: "formsgSiteLaunch"
support-1   |     meta: {
support-1   |       "domain": "isomer.gov.sg"
support-1   |     }
support-1   | { addAWSACMCertCAA: true, addLetsEncryptCAA: false }
support-1   | [
support-1   |   {
support-1   |     domain: 'google.com',
support-1   |     type: 'AAAA',
support-1   |     value: '2404:6800:4003:c11::64'
support-1   |   },
support-1   |   {
support-1   |     domain: 'google.com',
support-1   |     type: 'AAAA',
support-1   |     value: '2404:6800:4003:c11::65'
support-1   |   },
support-1   |   {
support-1   |     domain: 'google.com',
support-1   |     type: 'AAAA',
support-1   |     value: '2404:6800:4003:c11::71'
support-1   |   },
support-1   |   {
support-1   |     domain: 'google.com',
support-1   |     type: 'AAAA',
support-1   |     value: '2404:6800:4003:c11::8b'
support-1   |   }
support-1   | ]
support-1   | []
support-1   | { addAWSACMCertCAA: false, addLetsEncryptCAA: false }
support-1   | { addAWSACMCertCAA: false, addLetsEncryptCAA: false }
support-1   | []
support-1   | [08:59:33.272] INFO (56): Domain does not have any CAA records.
support-1   |     module: "formsgSiteLaunch"
support-1   |     meta: {
support-1   |       "domain": "isomer.gov.sg"
support-1   |     }
support-1   | [08:59:33.272] INFO (56): Domain does not have any CAA records.
support-1   |     module: "formsgSiteLaunch"
support-1   |     meta: {
support-1   |       "domain": "dontexist.sg"
support-1   |     }
support-1   | [08:59:33.272] INFO (56): Domain does not have any AAAA records.
support-1   |     module: "formsgSiteLaunch"
support-1   |     meta: {
support-1   |       "domain": "dontexist.sg"
support-1   |     }

feat: optimise local diff #1334

Copied over tests from #1172

  • Toggle feature flag is_local_diff_enabled for account being tested https://app.growthbook.io/features/is_local_diff_enabled
  • Create a page
  • Edit a page
  • Delete a page
  • Move a page
  • Try to create an RR and check that the diff shown is correct for all files (this tests that /compare works)
  • After creating the RR, navigate to the RR page and check that the diff shown is correct (this tests that /listFullReviewRequest works)
  • Approve and publish the RR
  • Check that diff is empty now (tests that master has been fastforwarded)
  • Finally, last sanity check that the diff still works after making another edit

Deploy Notes

Full Changelog: https://github.com/isomerpages/isomercms-backend/compare/v0.85.0..v0.86.0

harishv7 and others added 5 commits May 2, 2024 17:16
* fix: use faster diff-tree to get files changed between master and staging

* fix: GitFileSystemService tests

* fix: get all logs once instead of for each file and dedup queries for Users

* chore: fix tests

* chore(gitfileSystem): add trivial test cases

---------

Co-authored-by: Kishore <[email protected]>
@dcshzj dcshzj requested a review from a team May 13, 2024 09:53
@dcshzj dcshzj mentioned this pull request May 13, 2024
@dcshzj dcshzj merged commit fb96956 into master May 13, 2024
42 of 44 checks passed
@mergify mergify bot deleted the release_v0.86.0 branch May 13, 2024 10:10
This was referenced Jun 27, 2024
@dcshzj dcshzj mentioned this pull request Jun 27, 2024
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