-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
Custom and remote emojis #405
Conversation
hmmm I have no idea why the tests keep timing out |
@aumetra could you take a look at this PR when you've got some time pls? |
Will do on my commute home |
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.
Some initial comments. I'll look at it closer later
@aumetra thanks so much for these comments, I addressed everything now :) BTW there is one more aspect of this PR that I was thinking about - for the custom emoji table I decided to not save the domain for local emojis (it's NULL), but I saw that in other tables, for example for user accounts, the domain part is not nullable and saved even for local accounts. Should I change it accordingly for custom emojis? |
I don't think that's necessary here. We store the domains on accounts and users because of potential multi-tendant support (i.e. one instance with multiple domains). But not everyone will be allowed to add custom emoji anyway, just the moderators and admins of the overall system, so might as well not store the domain here. |
Right, then I'll leave it as it is |
@zeerooth would you mind rebasing this? I don't really have any more comments to this |
Merged main into it. Do you have any last things you wanna add before I hit the merge button? |
Not really, just that you're too quick, I merged it myself but then I try to push and see that you already pushed to this branch lol Thanks though, you can merge it now :) |
It's not ready to be merged or even reviewed, but I'd like to already open this draft PR to keep track of the progress and address potential feedback.
How it works:
Things that are already implemented and work:
Things that probably work but need to be tested:
but haven't tested it yet. Also who knows how it behaves with misskey or pleroma for example.it works!To be added:
Out of scope for now (I'll make an issue for it later)
Issue ref: #105