Skip to content

OF-1387 Add confirmation before deleting subscribers in pubsub admin UI #3215

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

Merged
guusdk merged 8 commits into from
Mar 25, 2026
+227 −98
Merged

OF-1387 Add confirmation before deleting subscribers in pubsub admin UI #3215

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
9134894
OF-1387: Add confirmation before deleting subscribers in pubsub-node-…
MilanTyagi2004 Mar 20, 2026
adb6a93
OF-1387: Implement dedicated confirmation page for PubSub subscriber …
MilanTyagi2004 Mar 21, 2026
8749a9e
trigger ci
MilanTyagi2004 Mar 21, 2026
23ea2c6
OF-1387: Fix missing affiliation in PubSub affiliate deletion, simpli…
MilanTyagi2004 Mar 22, 2026
cbdcf30
Merge branch 'main' into OF-1387-fix-subscriber-delete-confirmation
MilanTyagi2004 Mar 24, 2026
6d1155e
fix node affiliation deletion UI
MilanTyagi2004 Mar 25, 2026
137ecd3
Merge branch 'igniterealtime:main' into OF-1387-fix-subscriber-delete…
MilanTyagi2004 Mar 25, 2026
a0b1d19
Merge branch 'OF-1387-fix-subscriber-delete-confirmation' of https://…
MilanTyagi2004 Mar 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions i18n/src/main/resources/openfire_i18n.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2925,6 +2925,7 @@ pubsub.node.subscribers.expires=Expires
pubsub.node.subscribers.table.info=Current node subscribers are listed in the following table:
pubsub.node.subscribers.table.no_subscribers=The Node has no subscribers.
pubsub.node.subscribers.deleted=Deleted subscription owned by {0} from the node.
pubsub.node.subscribers.confirm_delete=Are you sure you want to delete this subscriber?

# PubSub service tab

Expand Down
2 changes: 1 addition & 1 deletion xmppserver/src/main/webapp/pubsub-node-subscribers.jsp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@
<c:param name="csrf" value="${csrf}" />
<c:param name="owner" value="${owner}"/>
</c:url>
<a href="${url}" title="<fmt:message key="global.click_delete" />">
<a href="${url}" title="<fmt:message key="global.click_delete" />" onclick="return confirm('<fmt:message key="pubsub.node.subscribers.confirm_delete" />');">
<img src="images/delete-16x16.gif" alt="">
</a>
</td>
Expand Down
Loading