-
Notifications
You must be signed in to change notification settings - Fork 1
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
[EASI-4473] Notification: Data Exchange Approach Completed #1209
Merged
OddTomBrooks
merged 19 commits into
NOREF/data_exchange_approach
from
feature/EASI-4473_notif_data_exchange_approach_complete
Jul 16, 2024
Merged
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
d5928f8
feat: added feature migration, data models, gql
OddTomBrooks c82e948
chore: updated sql queries
OddTomBrooks a3600a7
feat: several major improvements to first semi-functional data exchan…
OddTomBrooks aee9a0f
chore: updated postman collection to get all notifications
OddTomBrooks d01f101
chore: updated frontend gql and backend gql for user notification pre…
OddTomBrooks d52fdee
chore: added simple resolver helper to dispatch emails and email test…
OddTomBrooks db22226
chore: removed resolved question
OddTomBrooks a8dab47
chore: Reset all frontend files to main branch as we split the fronte…
OddTomBrooks 58a62c2
chore: reran gql gen
OddTomBrooks ac4cb5b
Merge branch 'main' into feature/EASI-4473_notif_data_exchange_approa…
OddTomBrooks a86a8c9
chore: email unit test for data exchange approach completed
OddTomBrooks d97ab3c
Merge branch 'feature/EASI-4473_notif_data_exchange_approach_complete…
OddTomBrooks 2e81438
chore: removed unnecessary Scan and Value methods
OddTomBrooks 834c34d
chore: added unit test for activity data exchange complete notification
OddTomBrooks e401f1d
chore: implemented resolver for data exchange approach completed noti…
OddTomBrooks 52ceb84
fix: updated various test definitions to match new user account prefe…
OddTomBrooks 8a9d37c
chore: removed outdated comment
OddTomBrooks 2e42900
chore: updated postman collection renaming markedCompletedBy -> marke…
OddTomBrooks cff7c48
chore: converted data exchange approach complete meta to id
OddTomBrooks File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,27 +10,27 @@ import ( | |
// is relevant to completing a Data Exchange Approach | ||
type DataExchangeApproachCompletedActivityMeta struct { | ||
ActivityMetaBaseStruct | ||
DataExchangeApproach *DataExchangeApproach `json:"dataExchangeApproach"` | ||
MarkedCompleteBy uuid.UUID `json:"markedCompleteBy"` | ||
DataExchangeApproachID uuid.UUID `json:"dataExchangeApproachID"` | ||
MarkedCompleteBy uuid.UUID `json:"markedCompleteBy"` | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note I think we probably want to the model_plan_id stored here too. |
||
|
||
// newDataExchangeApproachCompletedActivityMeta creates a new DataExchangeApproachCompletedActivityMeta | ||
func newDataExchangeApproachCompletedActivityMeta( | ||
dataExchangeApproach *DataExchangeApproach, | ||
dataExchangeApproachID uuid.UUID, | ||
markedCompleteBy uuid.UUID, | ||
) *DataExchangeApproachCompletedActivityMeta { | ||
version := 0 // iterate this if this type ever updates | ||
return &DataExchangeApproachCompletedActivityMeta{ | ||
ActivityMetaBaseStruct: NewActivityMetaBaseStruct(ActivityDataExchangeApproachCompleted, version), | ||
DataExchangeApproach: dataExchangeApproach, | ||
DataExchangeApproachID: dataExchangeApproachID, | ||
MarkedCompleteBy: markedCompleteBy, | ||
} | ||
} | ||
|
||
// NewDataExchangeApproachCompletedActivity creates a new Data Exchange Approach Complete type of Activity | ||
func NewDataExchangeApproachCompletedActivity( | ||
actorID uuid.UUID, | ||
dataExchangeApproach *DataExchangeApproach, | ||
dataExchangeApproachID uuid.UUID, | ||
markedCompleteBy uuid.UUID, | ||
) *Activity { | ||
return &Activity{ | ||
|
@@ -39,7 +39,7 @@ func NewDataExchangeApproachCompletedActivity( | |
EntityID: markedCompleteBy, | ||
ActivityType: ActivityDataExchangeApproachCompleted, | ||
MetaData: newDataExchangeApproachCompletedActivityMeta( | ||
dataExchangeApproach, | ||
dataExchangeApproachID, | ||
markedCompleteBy, | ||
), | ||
} | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think for this meta, we'd want both the ID, and the dataloader result to return the actually approach. Might be good to just add as a todo in the feature branch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(so that the front end can access the actual data if needed)