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

PEM-6153-fixed the fragment url to follow the actual content #4697

Merged
merged 11 commits into from
Dec 4, 2024

Conversation

nage1234
Copy link
Contributor

@nage1234 nage1234 commented Nov 13, 2024

Describe the Change

The change made as part of this PR is, when we load the packs doc page with fragment identifier (followed by "#"), it takes to the exact fragment in the "Additional Details"

Changed Pages

No access to netify, but, it doesnt impact on the design , in the packs page, go to pack detail page of any packs.
or directly hit the URL,

Preview URL

Jira Tickets

https://spectrocloud.atlassian.net/browse/PEM-6153

Backports

Can this PR be backported?

  • Yes. Remember to add the relevant backport labels to your PR.
  • No. Please leave a short comment below about why this PR cannot be backported.

Copy link

netlify bot commented Nov 13, 2024

Deploy Preview for docs-spectrocloud ready!

Name Link
🔨 Latest commit 041f1cc
🔍 Latest deploy log https://app.netlify.com/sites/docs-spectrocloud/deploys/675086cbbac510000860b37c
😎 Deploy Preview https://deploy-preview-4697--docs-spectrocloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@nageshsairam1610 nageshsairam1610 left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link
Contributor

@nageshsairam1610 nageshsairam1610 left a comment

Choose a reason for hiding this comment

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

minor change

@nageshsairam1610 nageshsairam1610 self-requested a review November 27, 2024 18:24
@karl-cardenas-coding karl-cardenas-coding added backport-version-4-0 Backport change to version 4.0 auto-backport Enable backport backport-version-3-4 Backport change to version 3.4 backport-version-4-1 Backport change to version 4.1 backport-version-4-2 Backport change to version 4.2 backport-version-4-3 Backport change to version 4.3 backport-version-4-4 Backport change to version 4.4 labels Dec 3, 2024
@karl-cardenas-coding karl-cardenas-coding changed the title PEM-6153-fixed the fragment url to follow the actual content WIP: PEM-6153-fixed the fragment url to follow the actual content Dec 4, 2024
@karl-cardenas-coding karl-cardenas-coding changed the title WIP: PEM-6153-fixed the fragment url to follow the actual content PEM-6153-fixed the fragment url to follow the actual content Dec 4, 2024
@karl-cardenas-coding karl-cardenas-coding merged commit db85b2d into master Dec 4, 2024
20 checks passed
@karl-cardenas-coding karl-cardenas-coding deleted the PEM-6153-nageswari branch December 4, 2024 18:12
vault-token-factory-spectrocloud bot pushed a commit that referenced this pull request Dec 4, 2024
…4697)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)
vault-token-factory-spectrocloud bot pushed a commit that referenced this pull request Dec 4, 2024
…4697)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)
vault-token-factory-spectrocloud bot pushed a commit that referenced this pull request Dec 4, 2024
…4697)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)
vault-token-factory-spectrocloud bot pushed a commit that referenced this pull request Dec 4, 2024
…4697)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)
vault-token-factory-spectrocloud bot pushed a commit that referenced this pull request Dec 4, 2024
…4697)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)
vault-token-factory-spectrocloud bot pushed a commit that referenced this pull request Dec 4, 2024
…4697)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)
vault-token-factory-spectrocloud bot pushed a commit that referenced this pull request Dec 4, 2024
…4697)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)
@vault-token-factory-spectrocloud
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
version-4-0
version-3-4
version-4-1
version-4-2
version-4-3
version-4-4
version-4-5

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

vault-token-factory-spectrocloud bot added a commit that referenced this pull request Dec 4, 2024
…4697) (#4862)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)

Co-authored-by: Nageswari Raghunathan <[email protected]>
vault-token-factory-spectrocloud bot added a commit that referenced this pull request Dec 4, 2024
…4697) (#4863)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)

Co-authored-by: Nageswari Raghunathan <[email protected]>
vault-token-factory-spectrocloud bot added a commit that referenced this pull request Dec 4, 2024
…4697) (#4864)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)

Co-authored-by: Nageswari Raghunathan <[email protected]>
vault-token-factory-spectrocloud bot added a commit that referenced this pull request Dec 4, 2024
…4697) (#4866)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)

Co-authored-by: Nageswari Raghunathan <[email protected]>
vault-token-factory-spectrocloud bot added a commit that referenced this pull request Dec 4, 2024
…4697) (#4865)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)

Co-authored-by: Nageswari Raghunathan <[email protected]>
vault-token-factory-spectrocloud bot added a commit that referenced this pull request Dec 4, 2024
…4697) (#4867)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)

Co-authored-by: Nageswari Raghunathan <[email protected]>
vault-token-factory-spectrocloud bot added a commit that referenced this pull request Dec 4, 2024
…4697) (#4868)

* PEM-6153-fixed the fragment url to follow the actual content

* ci: auto-formatting prettier issues

* changed the click as scrollIntoView

* reverting back to click

* fixed the fragment url for main Readme as well

* ci: auto-formatting prettier issues

* cleaned up

* ci: auto-formatting prettier issues

* addressed the review comments

* added fragment link support for h1 as well

---------

Co-authored-by: nage1234 <[email protected]>
Co-authored-by: Karl Cardenas <[email protected]>
(cherry picked from commit db85b2d)

Co-authored-by: Nageswari Raghunathan <[email protected]>
@vault-token-factory-spectrocloud
Copy link
Contributor

🎉 This PR is included in version 4.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link
Contributor

🤖 Starting the visual tests. This will take approximately an hour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Enable backport backport-version-3-4 Backport change to version 3.4 backport-version-4-0 Backport change to version 4.0 backport-version-4-1 Backport change to version 4.1 backport-version-4-2 Backport change to version 4.2 backport-version-4-3 Backport change to version 4.3 backport-version-4-4 Backport change to version 4.4 backport-version-4-5 Backport changes to the version-4-5 branch. javascript Pull requests that update Javascript code released visual-tests Trigger the visual tests workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants