Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

Modify the callbacks to use the new external param classes #1276

Closed
dperl-dls opened this issue Mar 19, 2024 · 2 comments · Fixed by #1366
Closed

Modify the callbacks to use the new external param classes #1276

dperl-dls opened this issue Mar 19, 2024 · 2 comments · Fixed by #1366
Assignees

Comments

@dperl-dls
Copy link
Collaborator

dperl-dls commented Mar 19, 2024

See #1092 and #1275

Acceptance Criteria

  • The callbacks use the new parameters and have no dependency on the old models
  • Tests all pass
  • In doing this think about callback structure, what parameters it needs and spin out new issues if there are ways to simplify
@DominicOram
Copy link
Collaborator

Depends on #1279 and #698

@DominicOram
Copy link
Collaborator

We think a M

@dperl-dls dperl-dls self-assigned this May 8, 2024
dperl-dls added a commit that referenced this issue May 9, 2024
dperl-dls added a commit that referenced this issue May 9, 2024
dperl-dls added a commit that referenced this issue May 9, 2024
dperl-dls added a commit that referenced this issue May 9, 2024
dperl-dls added a commit that referenced this issue May 16, 2024
dperl-dls added a commit that referenced this issue May 16, 2024
dperl-dls added a commit that referenced this issue May 16, 2024
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this issue Aug 23, 2024
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this issue Aug 23, 2024
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this issue Aug 23, 2024
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this issue Aug 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants