Skip to content

R/update (#48)#16

Draft
patroza wants to merge 4 commits into
mainfrom
bp-main
Draft

R/update (#48)#16
patroza wants to merge 4 commits into
mainfrom
bp-main

Conversation

@patroza
Copy link
Copy Markdown
Member

@patroza patroza commented May 9, 2026

  • update packages and settings

  • adopt code changes

  • config and package updates

  • Class->Struct conversions, Operations removal

  • lint fix

  • bs

  • e2e

  • fix ups

  • fix fe

  • bs

* update packages and settings

* adopt code changes

* config and package updates

* Class->Struct conversions, Operations removal

* lint fix

* bs

* e2e

* fix ups

* fix fe

* bs
@patroza
Copy link
Copy Markdown
Member Author

patroza commented May 9, 2026

@copilot fix conflicts, adopt changes/refactorings based on the latest commits in bp-main branch.
see effect-app source / history for additional help

Copilot AI and others added 3 commits May 9, 2026 11:49
Copy link
Copy Markdown

Copilot AI commented May 9, 2026

@copilot fix conflicts, adopt changes/refactorings based on the latest commits in bp-main branch.
see effect-app source / history for additional help

Done in 17f21a6. Merged origin/main into bp-main and resolved all conflicts:

  • Added new Blog feature (model, resources, controller, BlogPostRepo, pages) and Users feature (resource, controller, UserViews resolver) from main
  • Kept bp-main's S.Opaque refactoring, Operations removal, and package updates
  • Fixed Users.controllers.ts to map UserUserView using User.displayName() static method (required since User is now S.Opaque, not S.Class)
  • Fixed frontend/composables/client.ts to include UserViews layer from main while removing the OperationsClient export
  • Fixed pnpm-workspace.yaml allowBuilds placeholder values

API TypeScript check passes with no errors.

Copilot finished work on behalf of patroza May 9, 2026 12:01
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