patcon
Repos
622
Followers
273
Following
219

:milky_way: Open Source AI for large scale open ended feedback

419
102

An citizen-run online consultation experiment.

0
1

API for Canadian voter registration checks.

2
0

A demo for experimentation related to Toronto City Hall app.

2
0

city-specific django website for Toronto [Councilmatic]

7
7

Fork of 18F Identity Provider (IdP) - demo no longer live

1
0

Events

e2e: check that submit button isn't disabled before clicking.

Created at 8 hours ago
Attempt at fixing search-workflows.spec.js cypress test

Thanks @vd1992, was poking around that one too..! Documented in https://github.com/GCTC-NTGC/gc-digital-talent/issues/4035

Seems it might be related to ScrollToTop, which @esizer has raised as perhaps buggy (failure to clean-up). That sporadic error might resolve itself if we migrate to react-router (which has a component for that, presumably without this bug): https://github.com/GCTC-NTGC/gc-digital-talent/issues/3533#issuecomment-1244988917

Created at 9 hours ago
Clean up rounded edges on CallToAction

Adding radius actually causes more of a visual bug.

@esizer sorry, I think I resolved the visual bug you're referring to, no? That's what the change to the hydrogen config was (5=>6px).

Created at 9 hours ago

Bump @graphql-codegen/cli from 2.12.1 to 2.13.2 in /frontend

Bumps @graphql-codegen/cli from 2.12.1 to 2.13.2.


updated-dependencies:

  • dependency-name: "@graphql-codegen/cli" dependency-type: direct:development update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #4166 from GCTC-NTGC/dependabot/npm_and_yarn/frontend/graphql-codegen/cli-2.13.2

Bump @graphql-codegen/cli from 2.12.1 to 2.13.2 in /frontend

Created at 1 day ago
patcon delete branch dependabot/npm_and_yarn/frontend/graphql-codegen/cli-2.13.2
Created at 1 day ago
pull request closed
Bump @graphql-codegen/cli from 2.12.1 to 2.13.2 in /frontend

⚠️ Dependabot is rebasing this PR ⚠️

Rebasing might not happen immediately, so don't worry if this takes some time.

Note: if you make any changes to this PR yourself, they will take precedence over the rebase.


Bumps @graphql-codegen/cli from 2.12.1 to 2.13.2.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Created at 1 day ago
pull request closed
Bump engine.io and browser-sync in /tc-report/_site

Bumps engine.io and browser-sync. These dependencies needed to be updated together. Updates engine.io from 3.5.0 to 6.2.0

Updates browser-sync from 2.27.7 to 2.27.10

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) You can disable automated security fix PRs for this repo from the Security Alerts page.
Created at 1 day ago
Bump engine.io and browser-sync in /tc-report/_site

Closing as tc-report deps don't matter.

Created at 1 day ago
pull request closed
Bump follow-redirects from 1.14.5 to 1.15.2 in /tc-report/_site

Bumps follow-redirects from 1.14.5 to 1.15.2.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) You can disable automated security fix PRs for this repo from the Security Alerts page.
Created at 1 day ago
Bump follow-redirects from 1.14.5 to 1.15.2 in /tc-report/_site

Closing as tc-report deps don't matter.

Created at 1 day ago

fixup pages

Rename login command and add loginBySubject

Add new commands

New submit application workflow

Remove stray logging

increase colour contrast

Move toast before query wait

Update first_name labels to have consistent value

Update last_name labels to have consistent value

Update Create Account Form test use consistent first_name, last_name labels

Add view column, update email value

Add a test for talent cloud report page, before messing with it

Replace tc-report folder with contents of GCTC-NTGC/tc-report repo

Update htaccess files to point to moved static site

Fix htaccess config for moved static site

Update readme and delete now-unnecessary documentation

Temporarily disabled docker layer caching in github action.

fix spacing in about section

Fully removed docker layer caching.

Removed useful tasks from tc-report to root .vscode folder

Created at 1 day ago

Check for button existence before clicking.

Created at 1 day ago
Enforce descriptions on intl strings

No block, but soft -1.

I feel like we already over-use descriptions on principle.

  1. make its harder to de-dup single word translations into single string
  2. description are often just a verbose representation of hierarchy, which feels like it would be clearer as a custom id. "The header text of the foo section on the bar page" => bar.foo.header.

tl;dr - I already feel we use descriptions too often, with negligible value

Created at 1 day ago
Created at 3 days ago
issue comment
Remove dead code/routes

Would it make sense to mark either this or https://github.com/compdemocracy/polis/issues/263 as a dup, and bring relevant info to the open one?

Created at 4 days ago
issue comment
Remove slack routes from backend

Missed linking this earlier, but in case helpful: I had a few branching PRs that also removed some of this, and had passing e2e tests at the time https://github.com/compdemocracy/polis/pull/1234

And the cleaner diff, which is close to this one: https://github.com/patcon/polis/compare/263-remove-intercom...patcon:263-remove-slack

Created at 4 days ago
Attempt at fixing search-workflows.spec.js cypress test

New error! https://github.com/GCTC-NTGC/gc-digital-talent/actions/runs/3153065433/jobs/5129291173#step:8:532

Created at 4 days ago
Attempt at fixing search-workflows.spec.js cypress test

just gonna trigger re-runs a few times

good call! We can tag-team it until maybe 10-15 are done :)

Also, if there are still issues, we could prob be even more cautious by adding waits for each region added, instead of doing all at once, then waiting

Created at 5 days ago
pull request opened
Attempt at fixing search-workflows.spec.js cypress test

Closes: #4035

I suspect the issue may be that we just need to wait for the requests to run after each form field change, rather than blitz through a bunch of changes and then try to wait. It seems that there are maybe a few requests queued when we go to quick, and we don't properly wait for the final one before submitting the request

Created at 5 days ago
patcon create branch bug/4035-e2e-failures
Created at 5 days ago

Update first_name labels to have consistent value

Update last_name labels to have consistent value

Update Create Account Form test use consistent first_name, last_name labels

Temporarily disabled docker layer caching in github action.

Fully removed docker layer caching.

Merge pull request #4101 from GCTC-NTGC/bug/4056-name-labels

Update language used for first_name, last_name labels

Merge pull request #4108 from GCTC-NTGC/bug/4035-disable-docker-cache-action

Disable docker layer caching in github action

Merge branch 'main' into bug/4112-cta-corner-bevel

Created at 5 days ago
Bug - Improve unique email validation

Thanks a million @esizer! I think this may have resolved everything after https://github.com/GCTC-NTGC/gc-digital-talent/pull/4108 🎉

Created at 5 days ago

Temporarily disabled docker layer caching in github action.

Fully removed docker layer caching.

Merge pull request #4108 from GCTC-NTGC/bug/4035-disable-docker-cache-action

Disable docker layer caching in github action

Created at 5 days ago
patcon delete branch bug/4035-disable-docker-cache-action
Created at 5 days ago
pull request closed
Disable docker layer caching in github action

Spun out from: https://github.com/GCTC-NTGC/gc-digital-talent/issues/4035#issuecomment-1261116699 h/t @mnigh

This was causing e2e-tests to show as "skipped", and rarely showing the proper pass/fail status, which makes the e2e tests pretty useless.

Disabling docker layer cache for now, to avoid the github action "out of storage" error. We'll see in the github action run whether it makes the workflow run significantly longer (20-25 min seems to be regular).

Created at 5 days ago