fix: full domain should be updated when the form is saved on resource proxy page #509

New Issue

MrUnknownDE · 2026-04-05T17:14:20+02:00

MrUnknownDE commented

2026-04-05 17:14:20 +02:00

Originally created by @Fredkiss3 on 12/11/2025

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

I noticed a bug where when updating a resource domain, it would not be reflected in the infobox, and when going back and forth to the homepage, the value of the full domain would revert to the previous value.

The domain is correctly updated on the backend though, since we can see that if we refresh the page, it would reflect the actual value we updated.

The fix: calling updateResource() with the updated value, not the previous value of the resource.

screenshots

video

*Originally created by @Fredkiss3 on 12/11/2025* ## Community Contribution License Agreement By creating this pull request, I grant the project maintainers an unlimited, perpetual license to use, modify, and redistribute these contributions under any terms they choose, including both the AGPLv3 and the Fossorial Commercial license terms. I represent that I have the right to grant this license for all contributed content. ## Description I noticed a bug where when updating a resource domain, it would not be reflected in the infobox, and when going back and forth to the homepage, the value of the full domain would revert to the previous value. The domain is correctly updated on the backend though, since we can see that if we refresh the page, it would reflect the actual value we updated. The fix: calling `updateResource()` with the updated value, not the previous value of the resource. ## screenshots <table> <tr> <td align="center"> video </td> </tr> <tr> <td> <video src="https://github.com/user-attachments/assets/ae0fff1f-dfe1-4fdb-9325-15f620ca0a7d"></video> </td> </tr> </table>

MrUnknownDE closed this issue

2026-04-05 17:14:20 +02:00

Sign in to join this conversation.

Branches Tags

main

dev

dependabot/npm_and_yarn/dev-minor-updates-b4e5d6b9c5

revert-2766-feature/systemd-install-instructions

dependabot/npm_and_yarn/prod-patch-updates-05702d39f2

dependabot/npm_and_yarn/next-16.2.1

dependabot/npm_and_yarn/recharts-3.8.1

alerting-rules

private-site-ha

dependabot/docker/docker/library/node-25-slim

ssh

delete-account

msg-delivery

org-only-idp

cicd

patch

site-targets-auto-login

No Label

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/pangolin#509