Skip to content

Improve Readme + Logfire #554

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

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Improve Readme + Logfire #554

wants to merge 10 commits into from

Conversation

PrenSJ2
Copy link

@PrenSJ2 PrenSJ2 commented Nov 7, 2024

Technical Description

when validating ContractorModel in validation.py, we set last_updated (datetime). The problem is TC2 was not sending alast_updated field.

This is solved by using the requested_timestamp send by TC2 instead

Recreation steps

  • follow readme to setup
  • Setup socket-frontend too Improve Readme socket-frontend#357
  • once your socket integration in TC2 is connected, take your public key and set it in the socket-frontend public.html
  • in TC2, enable public tutor profiles
  • add the public profile label to 4 tutors
  • Settings > Approve Public Tutor Profiles
  • see the order of the contractors not move ( in the socket-frontend)

Testing

  • do above
  • see contractors move depending on last updated

@PrenSJ2 PrenSJ2 changed the title improve readme and makefirle improve readme and makefile Nov 7, 2024
@PrenSJ2 PrenSJ2 self-assigned this Nov 8, 2024
@PrenSJ2 PrenSJ2 changed the title improve readme and makefile improve readme and makefile + add Logfire Nov 8, 2024
@PrenSJ2 PrenSJ2 changed the title improve readme and makefile + add Logfire Fix Contractor's last_updated, + Logfire Nov 14, 2024
@PrenSJ2 PrenSJ2 changed the title Fix Contractor's last_updated, + Logfire Improve Readme + Logfire Nov 14, 2024
@PrenSJ2 PrenSJ2 removed their assignment Nov 14, 2024
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.

1 participant