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

Remove entire experience from logs #5101

Merged
merged 1 commit into from
Jul 18, 2024
Merged

Conversation

gilluminate
Copy link
Contributor

Description Of Changes

Experience object can get pretty huge and is mostly not helpful to log here.

Code Changes

  • clean up log line

Copy link

vercel bot commented Jul 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
fides-plus-nightly ⬜️ Ignored (Inspect) Jul 18, 2024 6:27pm

Copy link

cypress bot commented Jul 18, 2024

Passing run #8994 ↗︎

0 4 0 0 Flakiness 0
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

Merge e5e9566 into 51d7b88...
Project: fides Commit: 233fa037a6 ℹ️
Status: Passed Duration: 00:35 💡
Started: Jul 18, 2024 6:39 PM Ended: Jul 18, 2024 6:39 PM

Review all test suite changes for PR #5101 ↗︎

@gilluminate gilluminate merged commit 2c99458 into main Jul 18, 2024
13 checks passed
@gilluminate gilluminate deleted the nextjs-fidesjs-crash-issue branch July 18, 2024 18:40
Copy link

cypress bot commented Jul 18, 2024

Passing run #8998 ↗︎

0 4 0 0 Flakiness 0
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

Remove entire `experience` from logs (#5101)
Project: fides Commit: 2c99458620
Status: Passed Duration: 00:33 💡
Started: Jul 18, 2024 6:50 PM Ended: Jul 18, 2024 6:51 PM

Review all test suite changes for PR #5101 ↗︎

lucanovera pushed a commit that referenced this pull request Jul 18, 2024
@cypress cypress bot mentioned this pull request Jul 18, 2024
34 tasks
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.

2 participants