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

refactor: move sentry to platform #3451

Merged
merged 3 commits into from Nov 1, 2023

Conversation

nivedin
Copy link
Member

@nivedin nivedin commented Oct 12, 2023

Closes HFE-230 HFE-282

Description

This PR moves the sentry from hoppscotch-common to platform.

Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

@jamesgeorge007
Copy link
Member

Updates to empty states were made in #3452.

@liyasthomas
Copy link
Member

@nivedin revert this commit: 6dfca81

@nivedin
Copy link
Member Author

nivedin commented Oct 16, 2023

@nivedin revert this commit: 6dfca81

reverted this commit

@jamesgeorge007
Copy link
Member

jamesgeorge007 commented Oct 16, 2023

Are we getting this out with the next point release or the major one? Could you update the base branch accordingly?

@nivedin nivedin changed the base branch from release/2023.8.2 to release/2023.8.3 October 17, 2023 13:00
@jamesgeorge007
Copy link
Member

Could you resolve the conflicts?

@AndrewBastin AndrewBastin merged commit abd5288 into release/2023.8.3 Nov 1, 2023
1 check passed
@AndrewBastin AndrewBastin deleted the refactor/move-sentry-to-web branch November 1, 2023 12:47
AndrewBastin pushed a commit to AndrewBastin/hoppscotch that referenced this pull request Nov 2, 2023
AndrewBastin pushed a commit to AndrewBastin/hoppscotch that referenced this pull request Nov 2, 2023
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.

None yet

4 participants