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

Feat/bucket storage #36

Merged
merged 18 commits into from
May 17, 2024
Merged

Feat/bucket storage #36

merged 18 commits into from
May 17, 2024

Conversation

luuh-oliveira
Copy link
Contributor

@luuh-oliveira luuh-oliveira commented May 16, 2024

Objetivo

Adição de funcionalidade de adição/busca de imagem de perfil de Usuário.

Implementação

Conexão com Storage da Azure Storage, adição de classe de configuração e service do storage, criação de endpoints de gravação e busca de imagem e alteração do workflow para uso de secrets.


🚀 This description was created by Ellipsis for commit a35d29c

Summary:

This PR integrates Azure Blob Storage for managing user profile images, updates relevant services, and modifies Maven and GitHub Actions configurations, including dynamic fetching of storageUrl.

Key points:

  • Integrated Azure Blob Storage for user profile image management
  • Added UserController endpoints for image upload and retrieval
  • Introduced AzureBlobService for blob operations
  • Updated UserService for image management
  • Modified Maven configuration for Azure Blob Storage dependency
  • Updated GitHub Actions workflow for handling secrets
  • Dynamic fetching of storageUrl using @Value in AzureBlobService

Generated with ❤️ by ellipsis.dev

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

❌ Changes requested. Reviewed everything up to 55cb73d in 5 minutes and 45 seconds

More details
  • Looked at 318 lines of code in 8 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_TFShJOpyLENXNKxp


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

4 days left in your free trial, upgrade for $20/seat/month or contact us.

.github/workflows/maven.yml Outdated Show resolved Hide resolved
src/main/java/api/educai/controllers/PostController.java Outdated Show resolved Hide resolved
src/main/java/api/educai/controllers/UserController.java Outdated Show resolved Hide resolved
src/main/java/api/educai/services/AzureBlobService.java Outdated Show resolved Hide resolved
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

❌ Changes requested. Incremental review on 1d10985 in 2 minutes and 19 seconds

More details
  • Looked at 71 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_PYx7AR9g6tlLJe3o


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

4 days left in your free trial, upgrade for $20/seat/month or contact us.

.github/workflows/maven.yml Outdated Show resolved Hide resolved
luuh-oliveira and others added 2 commits May 16, 2024 21:15
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@luuh-oliveira luuh-oliveira merged commit 446e792 into main May 17, 2024
0 of 2 checks passed
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.

3 participants