Skip to content

Commit

Permalink
Merge pull request #59 from EveCrystali/dev/refactor
Browse files Browse the repository at this point in the history
Dev/refactor
  • Loading branch information
EveCrystali authored Nov 20, 2024
2 parents eba9f7a + cbd758d commit fbae366
Show file tree
Hide file tree
Showing 13 changed files with 3 additions and 108 deletions.
2 changes: 1 addition & 1 deletion BackendNote/Controllers/NotesController.cs
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ public async Task<IActionResult> UpdateNote(string id, Note updatedNote)

try
{
await _notesService.UpdateAsync(id, updatedNote);
await _notesService.UpdateAsync(updatedNote.Id, updatedNote);

Check warning on line 125 in BackendNote/Controllers/NotesController.cs

View workflow job for this annotation

GitHub Actions / Qodana for .NET

Possible null reference argument for a parameter.

Possible null reference argument for parameter 'id' in 'BackendNote.Services.NotesService.UpdateAsync'
}
catch (Exception ex)
{
Expand Down
5 changes: 0 additions & 5 deletions Frontend/Views/Shared/_Layout.cshtml.css
Original file line number Diff line number Diff line change
Expand Up @@ -35,11 +35,6 @@ a {
box-shadow: 0 .25rem .75rem rgba(0, 0, 0, .05);
}

button.accept-policy {
font-size: 1rem;
line-height: inherit;
}

.footer {
position: absolute;
bottom: 0;
Expand Down
4 changes: 2 additions & 2 deletions Frontend/Views/Shared/_LoginPartial.cshtml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
@using Microsoft.AspNetCore.Mvc.TagHelpers
<ul class="navbar-nav">
<li class="nav-item" id="login-item" style="display: true;">
<li class="nav-item" id="login-item" style="display: inline;">
<a class="nav-link" asp-controller="auth" asp-action="login">Connexion</a>
</li>

<li class="nav-item" id="logout-item" style="display: false;">
<li class="nav-item" id="logout-item" style="display: none;">
<form class="form-inline" asp-controller="auth" asp-action="logout">
<button type="submit" class="nav-link btn btn-link">Se déconnecter</button>
</form>
Expand Down
27 changes: 0 additions & 27 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,31 +64,6 @@ services:
networks:
- elastic

logstash:
build:
context: ./logstash
dockerfile: logstash.Dockerfile
container_name: logstash_container
volumes:
- ./logstash/pipeline:/usr/share/logstash/pipeline
- ./logstash/config:/usr/share/logstash/config
ports:
- "5044:5044"
env_file:
- .env
depends_on:
elasticsearch:
condition: service_healthy
mongodb:
condition: service_healthy
healthcheck:
test: [ "CMD", "curl", "-f", "http://localhost:9600/_node/stats" ]
interval: 5s
timeout: 10s
retries: 5
networks:
- elastic

mongo-express:
image: mongo-express
container_name: mongo_express
Expand Down Expand Up @@ -172,8 +147,6 @@ services:
depends_on:
elasticsearch:
condition: service_healthy
logstash:
condition: service_healthy
mongodb:
condition: service_healthy
environment:
Expand Down
Empty file removed logstash/config/jvm.options
Empty file.
Empty file removed logstash/config/log4j2.properties
Empty file.
Empty file.
6 changes: 0 additions & 6 deletions logstash/config/logstash.yml

This file was deleted.

6 changes: 0 additions & 6 deletions logstash/config/pipelines.yml

This file was deleted.

Empty file removed logstash/config/startup.options
Empty file.
24 changes: 0 additions & 24 deletions logstash/logstash.Dockerfile

This file was deleted.

6 changes: 0 additions & 6 deletions logstash/logstash.yml

This file was deleted.

31 changes: 0 additions & 31 deletions logstash/pipeline/logstash.conf

This file was deleted.

0 comments on commit fbae366

Please sign in to comment.