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.

Sign up for GitHub

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

Jump to bottom

Add location id to the SA result #2473

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

Add location id to the SA result #2473

wants to merge 22 commits into from
+48 −6

Conversation

AAJELLAL
Copy link
Contributor

No description provided.

@AAJELLAL
SA results: change column names
940552a 
Signed-off-by: AAJELLAL <ali.ajellalod@gmail.com>
@AAJELLAL
fix style
ed65b6f 
Signed-off-by: AAJELLAL <ali.ajellalod@gmail.com>
@AAJELLAL
Merge branch 'main' of https://github.com/gridsuite/gridstudy-app int…
d2a620e 
…o add-locationId-to-sa-result
@AAJELLAL AAJELLAL requested a review from jamal-khey January 20, 2025 08:26
Slimane AMAR and others added 10 commits January 27, 2025 15:36
Merge branch 'main' of https://github.com/gridsuite/gridstudy-app int…
99fea90 
…o add-locationId-to-sa-result
Merge branch 'main' of https://github.com/gridsuite/gridstudy-app int…
5e439f3 
…o add-locationId-to-sa-result
Merge branch 'main' of https://github.com/gridsuite/gridstudy-app int…
b28b9cb 
…o add-locationId-to-sa-result
Merge branch 'main' of https://github.com/gridsuite/gridstudy-app int…
8cb13aa 
…o add-locationId-to-sa-result
Merge branch 'main' of https://github.com/gridsuite/gridstudy-app int…
db04101 
…o add-locationId-to-sa-result
@AAJELLAL
Add locationId column.
46118b5 
Signed-off-by: AAJELLAL <ali.ajellalod@gmail.com>
@AAJELLAL
Some cleanup.
6dbdd26 
Signed-off-by: AAJELLAL <ali.ajellalod@gmail.com>
@AAJELLAL
Merge branch 'add-locationId-to-sa-result' of https://github.com/grid…
8bfd551 
…suite/gridstudy-app into add-locationId-to-sa-result
@AAJELLAL
Merge branch 'main' of https://github.com/gridsuite/gridstudy-app int…
df3d822 
…o add-locationId-to-sa-result
@AAJELLAL
Merge branch 'main' into add-locationId-to-sa-result
95cfa9c
@AAJELLAL AAJELLAL requested review from SlimaneAmar and jamal-khey and removed request for jamal-khey February 25, 2025 09:33
souissimai
souissimai approved these changes Feb 26, 2025
View reviewed changes
Copy link
Contributor

@souissimai souissimai left a comment

Choose a reason for hiding this comment

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

tested

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
75.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SlimaneAmar SlimaneAmar SlimaneAmar approved these changes

@souissimai souissimai souissimai approved these changes

@jamal-khey jamal-khey Awaiting requested review from jamal-khey

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AAJELLAL @jamal-khey @SlimaneAmar @souissimai