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

koa-zod-router #729

Draft
wants to merge 24 commits into
base: master
Choose a base branch
from
Draft

koa-zod-router #729

wants to merge 24 commits into from

Conversation

hawkrives
Copy link
Member

@hawkrives hawkrives commented May 12, 2024

  • add tsconfig file
  • begin adding typescript stuff
  • tweak tsconfig
  • ignore ts detritus
  • add ts-eslint to eslint
  • migrate "presence" module to typescript
  • fix noPropertyAccessFromIndexSignature + eslint conflict
  • move ccc-lib/http to typescript
  • migrate ccc-lib to ts
  • migrating ccc-calendar to ts
  • fix no-unused-vars to only run the ts-eslint version
  • add local typedef for ical.js
  • rename core server and institution modules to .ts
  • add typescript definitions for koa modules
  • remove default-case override
  • reformat eslint.config.js
  • port ccc-server to ts
  • add package.json script to invoke tsc
  • move to sentry 8-rc1 for better koa integration
  • migrate feeds to typescript
  • migrate bonapp menus to typescript
  • use lodash-es
  • port carleton orgs to typescript
  • change lodash types
  • port News endpoints to ts
  • a few fixups to carleton's orgs endpoint
  • rename all remaining files to .ts
  • begin updating dockerfile for tsc invocation
  • run tsc during ci
  • ignore the dist/ folder for eslint
  • switch to ava; add typed-query-selector; switch to printWidth=100
  • some reformatting
  • migrate remaining project files to typescript
  • fix final run path in Dockerfile
  • add missing types folder to Dockerfile
  • build the project in CI, where appropriate
  • remove .tsbuildinfo file when invoking tsc to ensure that it outputs the dist folder
  • disallow JS files
  • avoid building twice during smoke tests
  • convert RSS timestamps to ISO
  • fix deprecatedWpJson function
  • fixes for wpjson feeds
  • set default streams sort back to ascending
  • update Broadcast Media zod type
  • add koa-zod-router library
  • migrate core server to koa-zod-router
  • wip menu migration to koa-zod-router
  • continue work on menus

@hawkrives hawkrives changed the base branch from master to hawken/tsc-on-js May 25, 2024 05:52
@hawkrives hawkrives force-pushed the hawken/koa-cash branch 2 times, most recently from 112523e to 8f1405c Compare May 25, 2024 18:10
Base automatically changed from hawken/tsc-on-js to master May 25, 2024 20:17
@hawkrives hawkrives changed the title hawken/koa cash koa-zod-router May 25, 2024
Copy link
Member

Choose a reason for hiding this comment

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

It's fine organizationally, but I want to ask if we necessarily need to know the origin country and city level? I'm asking because I think this is still a pattern of "we'd like to share data between these two institutions, but really they're separate institutions"

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.

None yet

2 participants