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

More logging/fixes #3364

Merged
merged 4 commits into from
Dec 6, 2024
Merged

More logging/fixes #3364

merged 4 commits into from
Dec 6, 2024

Conversation

hagen-danswer
Copy link
Contributor

Description

  • Caught edge case where some spaces didnt get correctly premission retrieved
  • handled case where users didnt had email and not id
  • Logging!

Backporting (check the box to trigger backport action)

Note: You have to check that the action passes, otherwise resolve the conflicts manually and tag the patches.

  • This PR should be backported (make sure to check that the backport attempt succeeds)

Copy link

vercel bot commented Dec 6, 2024

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

Name Status Preview Comments Updated (UTC)
internal-search ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 6, 2024 9:34pm

@hagen-danswer hagen-danswer added this pull request to the merge queue Dec 6, 2024
Merged via the queue into main with commit 53428f6 Dec 6, 2024
13 checks passed
aronszanto pushed a commit to aronszanto/onyx that referenced this pull request Dec 23, 2024
* More logging for external group syncing

* Fixed edge case where some spaces were not being fetched

* made refresh frequency for confluence syncs configurable

* clarity
moshe added a commit to moshe/danswer that referenced this pull request Jan 12, 2025
* commit '970320bd': (1066 commits)
  Persona / prompt hardening (onyx-dot-app#3375)
  Fix Confluence perm sync for cloud users (onyx-dot-app#3374)
  Add more logging + retries to teams connector (onyx-dot-app#3369)
  hide oauth sources (onyx-dot-app#3368)
  Dev Version Niceness
  Bugfix/log exit code (onyx-dot-app#3362)
  More logging/fixes (onyx-dot-app#3364)
  fix slackbot channel config nullable (onyx-dot-app#3363)
  first cut at slack oauth flow (onyx-dot-app#3323)
  Improved file handling (onyx-dot-app#3353)
  k (onyx-dot-app#3358)
  Change default log level back to info
  Increase vespa attempts on startup
  fixed test
  Add test for Confluence permissions
  explicitly set page and attachment restrictions and space keys
  tygod for tests
  Added filter to slim connector and logging for space permissions
  Fix SlackBot Page Bugs (onyx-dot-app#3354)
  Add persona stats (onyx-dot-app#3282)
  ...
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