Skip to content

Added tenenat key to log#56

Open
amal-k-soman-aot wants to merge 2 commits into
AOT-Technologies:formsflow-mergefrom
amal-k-soman-aot:formsflow-merge
Open

Added tenenat key to log#56
amal-k-soman-aot wants to merge 2 commits into
AOT-Technologies:formsflow-mergefrom
amal-k-soman-aot:formsflow-merge

Conversation

@amal-k-soman-aot

@amal-k-soman-aot amal-k-soman-aot commented Feb 13, 2025

Copy link
Copy Markdown

Added Tenant Log

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • New Query Runner (Data Source)
  • New Alert Destination
  • Other

Description

Added Tenant Log

Related Tickets & Documents

https://aottech.atlassian.net/browse/FWF-4298

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

image

Comment thread redash/utils/format.py
}


org_id = "default"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It might be better to change this from a Class variable to an Instance Variables

from rq import get_current_job

class CurrentJobFilter(logging.Filter):
org_id = "default"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It might be better to change this from a Class variable to an Instance Variables

Comment thread redash/utils/format.py
@@ -0,0 +1,36 @@
"""Module handle the console display formatting."""
import logging
from redash import settings

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Settings is being imported but not used.

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