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

[MBL-984] Setup CreateFlagging GraphQL Mutation #1858

Merged
merged 5 commits into from Sep 27, 2023

Conversation

scottkicks
Copy link
Contributor

@scottkicks scottkicks commented Sep 25, 2023

📲 What

As part of our Report Project initiative - Update our GraphSchema so that we can use the CreateFlagging mutation.
Latest PR for more context on this initiative

🤔 Why

We'll use this mutation to submit reports to the Trust & Safety team.
The submission view that will utilize this mutation will be built next.

🛠 How

  • update graphql-schema using our local bin/apollo-schema-download.sh script
  • create CreateFlagging.graphql in /mutations
  • create a new adapter file + tests
  • create a new input file + tests
  • update Service, ServiceType, and MockService

✅ Acceptance criteria

  • GraphAPI has been updated with the new mutation and all tests pass
  • The new CreateFlagging mutation is ready to be referenced in the app. This works is to expose the mutation. Implementation will happen next.

by running our bin/apollo-schema-download.sh script
updates GraphAPI.swift after building with the new mutation
@scottkicks scottkicks self-assigned this Sep 25, 2023
@scottkicks scottkicks added this to the release-5.11.0 milestone Sep 25, 2023
@scottkicks scottkicks marked this pull request as ready for review September 25, 2023 19:17
@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Merging #1858 (59c7301) into main (ae5ef71) will decrease coverage by 0.01%.
The diff coverage is 79.31%.

@@            Coverage Diff             @@
##             main    #1858      +/-   ##
==========================================
- Coverage   83.98%   83.98%   -0.01%     
==========================================
  Files        1283     1287       +4     
  Lines      116986   117044      +58     
  Branches    31111    31129      +18     
==========================================
+ Hits        98256    98302      +46     
- Misses      17643    17655      +12     
  Partials     1087     1087              
Files Coverage Δ
KsApi/ServiceType.swift 75.65% <ø> (ø)
...hAPI.CreateFlaggingInput+CreateFlaggingInput.swift 100.00% <100.00%> (ø)
...CreateFlaggingInput+CreateFlaggingInputTests.swift 100.00% <100.00%> (ø)
KsApi/mutations/inputs/CreateFlaggingInput.swift 100.00% <100.00%> (ø)
...pi/mutations/inputs/CreateFlaggingInputTests.swift 100.00% <100.00%> (ø)
KsApi/MockService.swift 13.22% <14.28%> (+<0.01%) ⬆️
KsApi/Service.swift 8.79% <0.00%> (-0.14%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@ifosli ifosli left a comment

Choose a reason for hiding this comment

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

Just a minor comment, otherwise looks good. (I don't 100% understand this, but it seems reasonable and I don't think it's worth me taking the time to really dig into it right now)

@@ -73,6 +73,10 @@ public protocol ServiceType {

func createBacking(input: CreateBackingInput) ->
SignalProducer<CreateBackingEnvelope, ErrorEnvelope>

/// Sends report project data to relevant to a specific project
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: This sentence doesn't make sense to me. Maybe just "Sends report project data for a specific project"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no prob! feel free to suggest smaller changes like this directly in your comment. GitHub lets us commit directly from comments which can be nice.

@nativeksr
Copy link
Collaborator

nativeksr commented Sep 27, 2023

1 Warning
⚠️ Big PR

Generated by 🚫 Danger

@scottkicks scottkicks merged commit 70dc8b3 into main Sep 27, 2023
7 checks passed
@scottkicks scottkicks deleted the create-flagging-mutation branch September 27, 2023 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants