Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix headings and reorg #66

Merged
merged 1 commit into from
Jan 6, 2025
Merged

fix headings and reorg #66

merged 1 commit into from
Jan 6, 2025

Conversation

lbadenhop
Copy link
Contributor

Thank you for your contribution to the Axway-Open-Docs repo.

Describe the changes

Enter a brief description of your changes here to communicate to the maintainers what you changed and why.

Checklist for contributors

Before submitting this PR, please make sure:

  • You have read the contribution guidelines
  • You have signed the Axway CLA
  • You have verified the technical accuracy of your change
  • You have verified that your change does not expose any sensitive information (passwords, keys, etc.)
  • You have followed the Markdown guidelines (unless this is is a Netlify CMS contribution)

Put an x in the boxes that apply. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your change.

Checklist for approvers

This section is to be filled out by project maintainers only.

Before approving this PR, please make sure it:

  • Does not have conflicts with the base branch
  • Is clear and complete
  • Is believed to be accurate (technical accuracy to be checked with an SME for PRs originating from outside R&D)
  • Follows Axway style
  • Follows Markdown guidelines
  • Follows best practices
  • Does not contain typos, grammatical errors, etc.
  • Does not expose any sensitive information
  • Passes the Markdown linter rules (automated via CI check)
  • Does not contain broken links
  • Is discoverable and navigable

Checklist for mergers

This section is to be filled out by project maintainers only.

Before merging this PR, please make sure:

  • You have applied a size label (unless this work is tracked in the ReadMeFirst JIRA project)
  • You have applied a production label if this needs a manual push to Zoomin
  • You have added it to the correct GitHub project
  • You have added it to the correct Sprint milestone
  • You have appended a fixes line to this description if this PR fixes a specific GitHub issue
  • You have added a new collection to Netlify CMS config (static/admin/config.js) if required (new doc sections)
  • You have updated the Zoomin classification file if required (new AMPC topics)

@docsbuilder
Copy link

Deploy preview is ready!

Name Link
🔨 Latest commit ffd9879
😎 Deploy preview http://PR-66.open-banking-docs.opendocs-builder.pcloud.axway.int

@lbadenhop lbadenhop requested a review from axway-akb January 3, 2025 22:07
@lbadenhop
Copy link
Contributor Author

@axway-akb - I'll update the classification file once the PR is approved. This will only allow "customer-only" to view the security guide content.

Copy link
Collaborator

@axway-akb axway-akb 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.

@lbadenhop lbadenhop merged commit 046b7a0 into master Jan 6, 2025
1 check passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants