Skip to content

test(js): Add more fields to Group fixture #12759

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

Merged
merged 4 commits into from
Apr 15, 2019

Conversation

billyvg
Copy link
Member

@billyvg billyvg commented Apr 12, 2019

So that this more closely matches prod data

@billyvg billyvg requested a review from a team April 12, 2019 18:07
Copy link
Contributor

@mitsuhiko mitsuhiko left a comment

Choose a reason for hiding this comment

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

can you also add platform?

@mitsuhiko
Copy link
Contributor

To clarify why see this PR: #12757

@billyvg
Copy link
Member Author

billyvg commented Apr 12, 2019

@mitsuhiko is that going to be different than project platform?

@mitsuhiko
Copy link
Contributor

@billyvg: yep. The group platform is the platform of the last event. The project platform is the icon flavour.

@billyvg billyvg requested a review from mitsuhiko April 12, 2019 19:16
@billyvg billyvg force-pushed the test/js/add-more-fields-to-group-fixture branch from eb54ce6 to 68814b6 Compare April 15, 2019 19:40
@billyvg billyvg merged commit 25d76e4 into master Apr 15, 2019
@billyvg billyvg deleted the test/js/add-more-fields-to-group-fixture branch April 15, 2019 20:23
jan-auer added a commit that referenced this pull request Apr 16, 2019
* master: (50 commits)
  fix(ui) Don't show save-org-search on event search (#12785)
  ref(ui): Remove some unnecessary index.jsx files (#12606)
  feat(app-platform): Analytics (#12718)
  ref(js): Remove ApiMixin (#12384)
  test(js): Silence project plugin console info spam (#12761)
  test(js): Move SaveSearchStore.reset() (#12769)
  test(js): Add more fields to Group fixture (#12759)
  feat(app-platform): Integration "Learn More" modal (#12638)
  feat(saved-searches) Move create saved search button to search bar. (#12781)
  ref(global-header): Remove dead code (#12767)
  ref(releases): Refactored Releases Serializers (#12535)
  feat(app-platform): Sort Integrations (#12697)
  ref(audit-log): Log sso config updates (#12744)
  ref(app-platform): New 'Open In' UI  (#12621)
  feat(events): Use SnubaEvent if option is turned on (#12594)
  feat(global-selection-header): show settings icon link in single project mode (#12772)
  refs(api): Consolidate all search backend code into `SnubaSearchBackend`
  fix(tests) Remove large snapshots (#12766)
  fix: Update symbolicator snapshots (#12710)
  ref: Upgrade semaphore (#12751)
  ...
@github-actions github-actions bot locked and limited conversation to collaborators Dec 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants