Skip to content

Conversation

@aaronjae22
Copy link
Collaborator

This revert the logs that I merged on PR #161 and test implementation from #158.

The implementation worked as expected.

We can now include logging.

image

I add loggings in Testbed with the following format

def index(request):
    logger = logging.getLogger(__name__) # This could be added at the beginning of the file
    logger.info("Schema Index main page accessed")
    ....
    ....
    if search_query:
        results_count = results.count()
        logger.info(f"Search performed: query='{search_query}', results_count={results_count}")

        if results_count == 0:
            logger.warning(f"Search returned no results: query='{search_query}'")

I tested our logging implementation in Staging. I needed to add some roles to certain service accounts and I did the same process for Production so when we merge everything should be working the same way as Staging.

@aaronjae22 aaronjae22 self-assigned this Dec 17, 2025
@aaronjae22 aaronjae22 merged commit ed6d03d into main Dec 17, 2025
8 checks passed
@aaronjae22 aaronjae22 deleted the logging/rever-testing-logs branch December 17, 2025 20:12
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