Skip to content

Make it harder to forget to submit targets/filters subform #2627

Open

Description

@david-crespo

@iliana reports (and I think @charliepark have discussed this before) that it feels like the firewall rules form is just ignoring your target because you typed it in but forgot to click Add Target. There is probably something we can do about this. It's probably too much to refuse to submit the overall form if there's unsubmitted stuff in the target or filter field, but we can probably warn the user somehow.

Image

Activity

  1. charliepark commented on Jan 3, 2025

    @charliepark
    Contributor

    I'm not totally against disabling the form if there's stray data. "You have an unsaved {target|host} filter; save or clear it to create this firewall rule."

  2. david-crespo commented on Jan 3, 2025

    @david-crespo
    CollaboratorAuthor

    Yeah, on second thought it’s rare enough that it doesn’t feel like too much friction.

  3. added this to the 13 milestone on Jan 6, 2025
  4. self-assigned this
    on Jan 6, 2025
  5. modified the milestones: 13, 14 on Feb 11, 2025
  6. modified the milestones: 14, 15 on May 12, 2025
  7. modified the milestones: 15, 16 on Jun 9, 2025
  8. david-crespo commented on Aug 4, 2025

    @david-crespo
    CollaboratorAuthor

    As noted in #2634 (comment), we are hoping to fix this by eliminating the subforms and making the minitable editable directly instead. That PR (which doesn't exist yet) can close this issue.

  9. modified the milestones: 16, 17 on Aug 18, 2025
  10. charliepark commented on Aug 25, 2025

    @charliepark
    Contributor

    #2878 is the PR for minitable editing. Adding it so the various dashboards showing linked PRs can 🤞 auto update.

  11. removed their assignment
    on Aug 25, 2025
  12. modified the milestones: 17, Unscheduled on Sep 23, 2025
  13. removed this from the Unscheduled milestone on Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions