Skip to content

Fixes to mentoring session fetching#29

Merged
illicitonion merged 2 commits into
mainfrom
mentoring-fixes
Oct 17, 2025
Merged

Fixes to mentoring session fetching#29
illicitonion merged 2 commits into
mainfrom
mentoring-fixes

Conversation

@illicitonion
Copy link
Copy Markdown
Member

  • Treat lack of permissions as a non-fatal error
  • Log less verbosely - right now we are logging a huge amount

These are separate commits, and the first commit is from #28 and can be ignored here.

@illicitonion illicitonion requested a review from AnnaFYZ October 16, 2025 16:02
This is currently super noisy, because column 5 is a drop-down so is
never considered empty.

Instead, only log if the first cell of the row is non-empty, and there
weren't enough columns for that row.
@illicitonion illicitonion merged commit f9a3344 into main Oct 17, 2025
3 checks passed
@illicitonion illicitonion deleted the mentoring-fixes branch October 17, 2025 15:24
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