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

Yadhap/cicd azure #777

Closed
wants to merge 26 commits into from
Closed

Yadhap/cicd azure #777

wants to merge 26 commits into from

Conversation

ydahal1
Copy link
Collaborator

@ydahal1 ydahal1 commented May 28, 2024

Description

Please provide a summary of the changes made and the issue it resolves. Include relevant context and any dependencies required for this change.

Fixes # (issue)

Developer's Checklist

Please select at least one

  • Bug fix (non-breaking change that resolves an issue)
  • Breaking change (enhancement, fix, or feature that alters existing functionality)
  • New feature (non-breaking change that adds functionality)
  • Vulnerability fix (package updates or CodeQL adjustments to enhance code security)
  • Documentation update (addition or update of documentation or helper text)
  • Other change (please explain in the comment section)

Code Quality ( All must be selected )

  • I have commented on my code, especially in complex areas
  • I have resolved any conflicts with the target branch
  • My changes do not generate new warnings
  • I have checked my code for any misspellings
  • I have ensured my code does not duplicate existing code unnecessarily

Documentation

  • No documentation changes were required
  • I have made corresponding changes to the documentation

Security

  • I have confirmed that all security checks (e.g., CodeQL) have passed
  • No user input validation was required for this change
  • All user inputs have appropriate validation, including reasonable character limits

UX

  • This change does not involve any updates to UX elements
  • Refreshing related pages results in a functional and logical state
  • Appropriate error messages are displayed when necessary

Testing

  • Existing unit tests pass locally with my changes
  • No new unit tests were necessary for my changes
  • I have added new unit tests for my changes

Reviewer Checklist

  • I have successfully pulled the branch into my local environment, initiated the project, and verified that all the checked items above have passed

ydahal1 and others added 26 commits May 3, 2024 10:48
Verified user input on several routes, as well as added a global XSS sanitizer on our express app to prevent XSS attacks
…ct updating issue, remvoed console.log that was added for debugging prviously
* Create docs

This is a workflow file that will build and publish a github pages site to a 'gh-pages' branch so github can build a public site for our Docs on push to main

* recovered files

* saving progress

* updates

* fix breaklines
Fixed asr domain and product related issues
…lly indicating JM now is stopped and in pending state
Update migration file to match table name with the model file
@ydahal1 ydahal1 closed this May 28, 2024
@ydahal1 ydahal1 deleted the yadhap/cicd-azure branch June 6, 2024 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants