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

Reactions revamp & profile widgets #3272

Merged
merged 65 commits into from
Jun 21, 2023
Merged

Reactions revamp & profile widgets #3272

merged 65 commits into from
Jun 21, 2023

Conversation

tadhgboyle
Copy link
Member

@tadhgboyle tadhgboyle commented Mar 2, 2023 •
edited
Loading

Includes:

  • Better reaction UI
  • Emoji-based built in reactions
  • Allow users to react multiple times to a post/wall post
  • AJAX reaction modal to view who reacted what to a given post
  • Profile widget introduction
  • Minecraft account profile widget
  • Reactions profile widget

image

image

image

image

@tadhgboyle tadhgboyle added this to the 2.2.0 milestone Mar 2, 2023
@tadhgboyle tadhgboyle force-pushed the feat/reactions-revamp-profile-widgets branch from e7b0a15 to 3ca3e3f Compare March 2, 2023 00:38
tadhgboyle added 22 commits March 2, 2023 21:16
@tadhgboyle
Render emojis in reaction list StaffCP
8c052f2
@tadhgboyle
wip minecraft account profile widget, sticky widgets on profile page
5e68508
@tadhgboyle
Add server name and IP to last seen
611803d
@tadhgboyle
remove
cb41d91
@tadhgboyle
rename to reaction score to match member list
d9b5ccc
@tadhgboyle
align reactions table contents center
5ee84fa
@tadhgboyle
fix
298d500
@tadhgboyle
wip - nicer abstraction, widgets never query db, skin 3d viewer (unde…
160b6c7 
…cided)
@tadhgboyle
wip - make reaction modals ajax, copy new reaction bar to profile posts
70f6188
@tadhgboyle
wip reaction modals
a99a5a5
@tadhgboyle
wip proper sorting
d519967
@tadhgboyle
wip modals
ea21033
@tadhgboyle
wip get reaction submission working on profile wall posts
32404c8
@tadhgboyle
add back helpful and creative default reactions
e822215
@tadhgboyle
reduce query amount on view topic
5faca3a
@tadhgboyle
remove debug statement
d6f5ccb
@tadhgboyle
wip
3fd4a9c
@tadhgboyle
wip
a8d7dec
@tadhgboyle
simplify ProfileWidget pages
9f3ac4b
@tadhgboyle
Merge branch 'develop' into feat/reactions-revamp-profile-widgets
d0f8748
@tadhgboyle
Revert forum dropdown
f6eda0c
@tadhgboyle
fix css syntax
366ba95
@tadhgboyle tadhgboyle force-pushed the feat/reactions-revamp-profile-widgets branch from ec47a2d to 67d1987 Compare June 13, 2023 03:17
@tadhgboyle
Merge remote-tracking branch 'origin/develop' into feat/reactions-rev…
0b51a97 
…amp-profile-widgets

# Conflicts:
#	modules/Core/includes/endpoints/ServerInfoEndpoint.php
#	modules/Forum/pages/forum/reactions.php
#	modules/Forum/pages/forum/view_topic.php
@tadhgboyle
wip
b1d6f20
partydragen
partydragen requested changes Jun 14, 2023
View reviewed changes
core/classes/Widgets/AbstractWidget.php Outdated Show resolved Hide resolved
core/classes/Widgets/AbstractWidget.php Outdated Show resolved Hide resolved
@tadhgboyle tadhgboyle marked this pull request as ready for review June 16, 2023 17:46
@tadhgboyle tadhgboyle requested review from a team and partydragen June 16, 2023 17:47
partydragen
partydragen previously approved these changes Jun 19, 2023
View reviewed changes
@partydragen
Copy link
Member

Someone else should approve too not just me xD

supercrafter100
supercrafter100 reviewed Jun 19, 2023
View reviewed changes
Copy link
Member

@supercrafter100 supercrafter100 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haven't finished completely yet. Will hopefully finish the review soon

core/classes/DTO/Reaction.php Show resolved Hide resolved
core/classes/Misc/ProfilePostReactionContext.php Show resolved Hide resolved
modules/Core/includes/endpoints/ServerInfoEndpoint.php Show resolved Hide resolved
samerton
samerton requested changes Jun 20, 2023
View reviewed changes
core/classes/DTO/Reaction.php Outdated Show resolved Hide resolved
modules/Core/classes/Events/UserReactionAddedEvent.php Outdated Show resolved Hide resolved
modules/Core/pages/panel/reactions.php Outdated Show resolved Hide resolved
modules/Core/widgets/ReactionsProfileWidget.php Outdated Show resolved Hide resolved
modules/Core/widgets/ReactionsProfileWidget.php Outdated Show resolved Hide resolved
modules/Core/widgets/ReactionsProfileWidget.php Outdated Show resolved Hide resolved
modules/Forum/pages/forum/view_topic.php Show resolved Hide resolved
@samerton samerton merged commit b13c29e into develop Jun 21, 2023
@samerton samerton deleted the feat/reactions-revamp-profile-widgets branch June 21, 2023 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@supercrafter100 supercrafter100 supercrafter100 left review comments

@samerton samerton samerton approved these changes

@partydragen partydragen Awaiting requested review from partydragen

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
2.2.0
Development

Successfully merging this pull request may close these issues.
4 participants
@tadhgboyle @partydragen @samerton @supercrafter100