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

Unable to rename graphicOverview #8442

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tylerjmchugh
Copy link
Contributor

This PR arose from the HNAP PR 403 where the graphic overviews could not be updated with a new name. It was suggested that iso19139 could also use these changes. I am unable to test this however as there does not appear to be an edit option for graphic overviews on iso19139 like there is on HNAP.

Checklist

  • I have read the contribution guidelines
  • Pull request provided for main branch, backports managed with label
  • Good housekeeping of code, cleaning up comments, tests, and documentation
  • Clean commit history broken into understandable chucks, avoiding big commits with hundreds of files, cautious of reformatting and whitespace changes
  • Clean commit messages, longer verbose messages are encouraged
  • API Changes are identified in commit messages
  • Testing provided for features or enhancements using automatic tests
  • User documentation provided for new features or enhancements in manual
  • Build documentation provided for development instructions in README.md files
  • Library management using pom.xml dependency management. Update build documentation with intended library use and library tutorials or documentation

@ianwallen ianwallen added this to the 4.4.6 milestone Oct 16, 2024
@josegar74
Copy link
Member

With iso19139 in main branch to get the edit button in the thumbnails is required to use in the sidePanel element:

 <directive data-gn-onlinesrc-list=""/>
    <view name="default" default="true"
          class="gn-label-above-input gn-indent-bluescale">
      <sidePanel>
          <directive data-gn-onlinesrc-list=""/>
...

I can not reproduce the issue in main, where I could update the resource name properly.

Pending to verify it 4.2.x branch.

@josegar74 josegar74 modified the milestones: 4.4.6, 4.4.7 Oct 17, 2024
@tylerjmchugh
Copy link
Contributor Author

tylerjmchugh commented Oct 17, 2024

With iso19139 in main branch to get the edit button in the thumbnails is required to use in the sidePanel element:

 <directive data-gn-onlinesrc-list=""/>
    <view name="default" default="true"
          class="gn-label-above-input gn-indent-bluescale">
      <sidePanel>
          <directive data-gn-onlinesrc-list=""/>
...

I can not reproduce the issue in main, where I could update the resource name properly.

Pending to verify it 4.2.x branch.

@josegar74 Tested on 4.2.x and I cannot reproduce there either. The file name seems to be updated correctly.

So this PR is not required then?

@ianwallen
Copy link
Contributor

ianwallen commented Oct 17, 2024

@josegar74, @tylerjmchugh
If the changes work with or without these changes I suggest to still apply the changes because

  • it is the same logic as HNAP so it would avoid confusion.
  • it uses similar logic to onlinesrc-add.xsl so it would avoid confusion.

So by applying the changes, it would make the code more consistent.

@tylerjmchugh
Copy link
Contributor Author

tylerjmchugh commented Oct 17, 2024

@josegar74, @tylerjmchugh If the changes work with or without these changes I suggest to still apply the changes because

  • it is the same logic as HNAP so it would avoid confusion.
  • it uses similar logic to onlinesrc-add.xsl so it would avoid confusion.

So by applying the changes, it would make the code more consistent.

Tested and confirmed that resources can be renamed with and without the changes on main and 4.2.x. Applying the changes anyways should not cause issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants