Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

jsandoval81

jsandoval81 created a review on a pull request on csg-org/CompactConnect
Looking solid 👍 This was a mostly code review. I'll take a deeper dive in the UI on the next pass.

View on GitHub

jsandoval81 created a review comment on a pull request on csg-org/CompactConnect
Minor: Move this to the `// Reds` section. Also, is this just a new color in the designs? Or did you have to create it to meet a11y? Ideally, we could use the same red we've been using for error...

View on GitHub

jsandoval81 created a review comment on a pull request on csg-org/CompactConnect
Minor: The pattern up to now has been to declare computeds like stores, compact, user permissions & "high-level" things near the top of the computed block; as a means of making it easier to find in...

View on GitHub

jsandoval81 created a review comment on a pull request on csg-org/CompactConnect
Does this need the `compact` param?

View on GitHub

jsandoval81 created a review comment on a pull request on csg-org/CompactConnect
Minor: Prefer color variables.

View on GitHub

jsandoval81 created a review comment on a pull request on csg-org/CompactConnect
Same as earlier comment - existing patterns seem to be just as effective, and not require `any`. Also, making sure `this.watchFormInputs()` is excluded on purpose.

View on GitHub

jsandoval81 created a review comment on a pull request on csg-org/CompactConnect
`isLoading` is probably meant to observe a different store here.

View on GitHub

jsandoval81 created a review comment on a pull request on csg-org/CompactConnect
Should these be removed?

View on GitHub

jsandoval81 created a review comment on a pull request on csg-org/CompactConnect
Could this just be: `this.formData = reactive({...});`? This is what we do everywhere else and it would allow us to not need to declare the `any` type. Also, I noticed we aren't initiating `this...

View on GitHub

jsandoval81 created a review comment on a pull request on csg-org/CompactConnect
Minor: Prefer well-described variable names; e.g. `file` (?) over `f`. Or looking closer, it may be that the array is an array of military affiliations; e.g. `affiliation` or `militaryAffiliations`.

View on GitHub

jsandoval81 created a comment on a pull request on csg-org/CompactConnect
@jlkravitz This is ready for your review.

View on GitHub

jsandoval81 opened a pull request on csg-org/CompactConnect
path-to-regexp dependency update
### Requirements List - `yarn install --ignore-engines` ### Description List - Address https://github.com/csg-org/CompactConnect/security/dependabot/63 ### Testing List - `yarn test:unit:a...
jsandoval81 created a branch on InspiringApps/JCC-CompactConnect

frontend/dependency-updates - A system to share professional licensure data between state licensing boards for participating in occupational licensure compacts.

jsandoval81 opened an issue on csg-org/CompactConnect
Frontend dependabot updates
jsandoval81 opened a pull request on csg-org/CompactConnect
Frontend/Fix pw input validation messages
### Requirements List - _None_ ### Description List - Move message augmentation to after message evaluation - Update frontend to match latest backend pw requirements (length only) - Add test...
jsandoval81 deleted a branch InspiringApps/JCC-CompactConnect

frontend/dependency-updates

jsandoval81 created a branch on InspiringApps/JCC-CompactConnect

frontend/pw-input-fix - A system to share professional licensure data between state licensing boards for participating in occupational licensure compacts.

jsandoval81 opened a pull request on csg-org/CompactConnect
Frontend/permission api updates
### Requirements List - _None_ ### Description List - Change compact-level `read` permission to `readPrivate` - Add state-level `readPrivate` permission - Omit compact permission in server p...
jsandoval81 pushed 4 commits to frontend/permission-api-updates InspiringApps/JCC-CompactConnect
  • Frontend/hifi style (#419) ### Requirements List - _None_ ### Description List - Swapped out the existing lo-fi... b9bdfb3
  • Feat/Transaction History Processing Workflow with Email Notifications (#420) This PR adds a new workflow for process... 455f758
  • Remove mock resources from persistent stack (#421) This is part two of the mock resource cleanup. Now that the API a... 7ca275d
  • Merge branch 'development' into frontend/permission-api-updates c2d6ce5

View on GitHub

jsandoval81 pushed 1 commit to frontend/permission-api-updates InspiringApps/JCC-CompactConnect
  • WIP: Permission API updates - Handle board ed conditional case - @todo: Merge latest development - @todo: Smoke test ... 6ae444c

View on GitHub

jsandoval81 created a branch on InspiringApps/JCC-CompactConnect

frontend/permission-api-updates - A system to share professional licensure data between state licensing boards for participating in occupational licensure compacts.

jsandoval81 created a comment on a pull request on csg-org/CompactConnect
@jlkravitz This is ready for your review.

View on GitHub

jsandoval81 pushed 1 commit to frontend/hifi-style InspiringApps/JCC-CompactConnect
  • Re-enable compact selector 99f0b14

View on GitHub

jsandoval81 pushed 1 commit to frontend/hifi-style InspiringApps/JCC-CompactConnect
  • Hi-fi updates - Added compact selector to new nav - Added off-click for nav - Privilege purchase flow minor tuning up... 5f94458

View on GitHub

jsandoval81 pushed 4 commits to frontend/hifi-style InspiringApps/JCC-CompactConnect
  • Feat/auth tweaks (#388) ### Requirements List - Python 3.12+ - A working sandbox account, once you have access you... 5977b08
  • Feat/ssn table (#411) 8ff3a18
  • Feat/read access updates (#387) fb49f60
  • Merge branch 'development' into frontend/hifi-style fc75abd

View on GitHub