Skip to content

[Feature] URL Shortener - Misc. Updates#26

Merged
elliot-huffman merged 6 commits intomainfrom
pasha-sus-update
Oct 14, 2025
Merged

[Feature] URL Shortener - Misc. Updates#26
elliot-huffman merged 6 commits intomainfrom
pasha-sus-update

Conversation

@pasha-zayko
Copy link
Contributor

Adding new fields available in the database record

Adjusting auth.id call format to match the rest of the spec file

Adding new fields available in the database record

Adjusting auth.id call format to match the rest of the spec file
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@pasha-zayko pasha-zayko self-assigned this Sep 4, 2025
@pasha-zayko pasha-zayko added the enhancement New feature or request label Sep 4, 2025
Copy link
Contributor

@elliot-huffman elliot-huffman left a comment

Choose a reason for hiding this comment

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

Please update this PR to be compliant with the new OpenAPI 3.1 standard that was merged.

@elliot-huffman elliot-huffman changed the title Updating spec to match code changes [Feature] URL Shortener - Misc. Updates Oct 9, 2025
@pasha-zayko pasha-zayko marked this pull request as draft October 9, 2025 19:11
Enhancing definitions with example sections too fully match OpenApi spec v3.1
@pasha-zayko pasha-zayko requested a review from Copilot October 9, 2025 20:35
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • src/urlShortener/TypeScript/package-lock.json: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@pasha-zayko pasha-zayko marked this pull request as ready for review October 9, 2025 20:36
Fixing typo
Copy link
Contributor

@elliot-huffman elliot-huffman 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

@elliot-huffman elliot-huffman merged commit edca7ea into main Oct 14, 2025
6 checks passed
@elliot-huffman elliot-huffman deleted the pasha-sus-update branch October 14, 2025 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants