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.

Sign up for GitHub

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

Jump to bottom

Make sure the default recordingid / authkey() business is properly documented #3798

Closed
teh-cmc opened this issue Oct 11, 2023 · 0 comments · Fixed by #3799
Closed

Make sure the default recordingid / authkey() business is properly documented #3798

teh-cmc opened this issue Oct 11, 2023 · 0 comments · Fixed by #3799
Assignees
@teh-cmc
Labels
📖 documentation Improvements or additions to documentation 🐍 Python API Python logging API
Milestone
0.9.1

Comments

@teh-cmc
Copy link
Member

teh-cmc commented Oct 11, 2023

Footgun, see:
@teh-cmc teh-cmc added 📖 documentation Improvements or additions to documentation 🐍 Python API Python logging API labels Oct 11, 2023
@teh-cmc teh-cmc added this to the 0.9.1 milestone Oct 11, 2023
@teh-cmc teh-cmc self-assigned this Oct 11, 2023
This was referenced Oct 11, 2023
Open
Merged
teh-cmc added a commit that referenced this issue Oct 12, 2023
@teh-cmc
Add scary warning that default recording_id = authkey (#3799)
2cada94 
Rendered:

![image](https://github.com/rerun-io/rerun/assets/2910679/645c216b-488a-4ef5-9627-e476bb29e8e7)

- Fixes #3798
emilk pushed a commit that referenced this issue Oct 12, 2023
@teh-cmc @emilk
Add scary warning that default recording_id = authkey (#3799)
b02bb5e 
Rendered:

![image](https://github.com/rerun-io/rerun/assets/2910679/645c216b-488a-4ef5-9627-e476bb29e8e7)

- Fixes #3798
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@teh-cmc teh-cmc

Labels
📖 documentation Improvements or additions to documentation 🐍 Python API Python logging API
Projects
None yet
Milestone
0.9.1
Development

Successfully merging a pull request may close this issue.

Add scary warning that default recording_id = authkey rerun-io/rerun
1 participant
@teh-cmc