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

Reporting Module: Add Basic Model/Query/Mutation #1357

Merged
merged 9 commits into from
Nov 8, 2023

Conversation

thenav56
Copy link
Member

@thenav56 thenav56 commented Aug 23, 2023

Addresses

Changes

  • Add Models for AnalysisReport
  • Add Query for AnalysisReport
  • Add Mutation for AnalysisReport

This PR doesn't introduce any:

  • temporary files, auto-generated files or secret keys
  • n+1 queries
  • flake8 issues
  • print
  • typos
  • unwanted comments

This PR contains valid:

  • tests
  • permission checks (tests here too)
  • translations

@thenav56 thenav56 force-pushed the feature/report-module branch 5 times, most recently from 8a620ff to 42ca11a Compare September 4, 2023 08:23
@thenav56 thenav56 force-pushed the feature/report-module branch from 1bc55f5 to 815d711 Compare September 6, 2023 08:52
@thenav56 thenav56 force-pushed the feature/report-module branch from 1c0657e to 542efed Compare September 22, 2023 06:24
@AdityaKhatri AdityaKhatri changed the title Add Basic Model/Query/Mutation Reporting Module: Add Basic Model/Query/Mutation Sep 26, 2023
@thenav56 thenav56 force-pushed the feature/report-module branch 2 times, most recently from 6c643cc to b2bfb57 Compare September 28, 2023 06:08
@thenav56 thenav56 force-pushed the feature/report-module branch 4 times, most recently from 04cbafc to e4def4b Compare October 6, 2023 05:13
@thenav56 thenav56 force-pushed the feature/report-module branch 3 times, most recently from 51560b7 to 1072460 Compare October 31, 2023 09:43
@codecov
Copy link

codecov bot commented Oct 31, 2023

We're currently processing your upload. This comment will be updated when the results are available.

@AdityaKhatri AdityaKhatri force-pushed the feature/report-module branch from 1072460 to 2eee786 Compare October 31, 2023 10:41
@AdityaKhatri AdityaKhatri marked this pull request as ready for review November 2, 2023 05:46
Copy link
Contributor

@Rup-Narayan-Rajbanshi Rup-Narayan-Rajbanshi left a comment

Choose a reason for hiding this comment

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

LGTM

@AdityaKhatri AdityaKhatri force-pushed the feature/report-module branch 2 times, most recently from 07aa5b5 to d3c1b04 Compare November 8, 2023 08:21
- Remove contentStyle from container
- Rename contentStyle to style inside each content configuration
- Add style to image and url inside content configuration based on updated schema in https://github.com/orgs/the-deep/projects/2/views/12?pane=issue&itemId=34738356
- Update content style of heading to use HeadingContentStyle instead of TextStyle (updated in schema)
- Add clientId to container
- Add content_type in AnalysisReportContainerType node
- Add user-resource field to Report Node
- Add latest snapshot info in Report Node
@AdityaKhatri AdityaKhatri merged commit 5cd5325 into develop Nov 8, 2023
3 of 4 checks passed
AdityaKhatri and others added 3 commits November 8, 2023 14:07
- Add basic test cases
- Add enable_publicly_viewable_analysis_report_snapshot in Project for
  global switch
@AdityaKhatri AdityaKhatri deleted the feature/report-module branch November 8, 2023 08:22
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.

4 participants