Skip to content

WIP#107

Draft
kigiri wants to merge 7 commits into
masterfrom
WIP-local-env
Draft

WIP#107
kigiri wants to merge 7 commits into
masterfrom
WIP-local-env

Conversation

@kigiri
Copy link
Copy Markdown
Member

@kigiri kigiri commented Apr 3, 2026

No description provided.

Comment thread api/routes.ts Outdated
output: deploymentOutput,
description: 'Create a new deployment',
}),
'POST/api/deployment/local': route({
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

add the port or a slug to identify the deployment

  'POST/api/deployment/local': route({
     ....
      if (!DeploymentsCollection.get('dev')) {
        await DeploymentsCollection.insert({
          projectId: 'local',
          url: input.port, // or slug
          .....
        })
      }
    },
    input: OBJ({ endpoint: optional(STR('Full href of the SQL endpoint')),port: NUM() }), // or slug 
  }),

Comment thread api/schema.ts Outdated
STR,
} from '@01edu/api/validator'
import { createCollection } from '/api/lib/json_store.ts'
import { isLocal } from '/api/lib/env.ts'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this is not used

Copy link
Copy Markdown
Member

@abdotop abdotop left a comment

Choose a reason for hiding this comment

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

Juste fix lint and conflict

@kigiri kigiri force-pushed the WIP-local-env branch 2 times, most recently from 1ec6bb2 to b77df56 Compare April 9, 2026 13:47
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.

2 participants