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.

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

Add contributor alias to blost post #550

Merged
merged 1 commit into from
Dec 29, 2024

Conversation

jauggy
Copy link
Member

@jauggy jauggy commented Dec 29, 2024

This feature is useful for people who have a contributor name that is different from their in-game name. Currently when you make a blogpost, it uses your ingame name. This feature allows you to set a contributor alias (display name) to show for your blog post. The userid is still saved in the database so if there is any abuse (like impersonation), an admin would know.

Screenshot 2024-12-29 132602
Screenshot 2024-12-29 174315

Testing

Local testing

mix ecto.migrate

This will create a new column in the database.

Login to website and go to Blog > Blog Admin

Fill in the form. You will notice the Post button is disabled. That's because you must attach a tag. If there are no tags, go to the Tags tab and create one. Then you can press post.

Check the post appears.

You can retest using a different author alias or even a blank contributor alias (will use your in-game name).

Testing discord integration

Currently blog posts have a post in discord too. I don't know how to test this manually. However, I've added some unit tests.

@jauggy jauggy changed the title Add author alias to blost post Add contributor alias to blost post Dec 29, 2024
@jauggy jauggy marked this pull request as ready for review December 29, 2024 06:46
@L-e-x-o-n L-e-x-o-n merged commit f4e1556 into beyond-all-reason:master Dec 29, 2024
3 checks passed
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