-
Notifications
You must be signed in to change notification settings - Fork 406
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
Lexicon: Replace all user parameters with actor or repo #655
Conversation
"parameters": { | ||
"type": "params", | ||
"required": ["author"], | ||
"required": ["actor"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took a bit of liberty here & changed it to actor
even tho it wasn't ambiguous like user
. Let me know if you disagree
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Smooth!
Ooh, found one little thing I wonder if we oughtta tidy-up while we're in here. There's a field called |
I just pushed up that change I mentioned above— we can always revert if we'd rather not include it 👍 |
Nope I like it! & now that we have formats, it's totally clear what the value is supposed to be by looking at the schema 👍 |
7832eb2
to
4c3a1ee
Compare
Remove vague "user" references.
If in the
com.atproto
namespace, we use "repo"If in the
app.bsky
, we use "actor"Closes #591
Closes #548