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

Show newly deprecated #1165

Merged
merged 17 commits into from Nov 3, 2021
Merged

Show newly deprecated #1165

merged 17 commits into from Nov 3, 2021

Conversation

joelit
Copy link
Contributor

@joelit joelit commented May 20, 2021

This is an otherwise ready PR - however, the labels for new concepts replacing deprecated ones is not implemented since it caused a significant regression in performance. A follow-up fix will come later.

@joelit joelit requested a review from kouralex May 20, 2021 14:35
@codecov
Copy link

codecov bot commented May 20, 2021

Codecov Report

Merging #1165 (e7f3bef) into master (d12c2d3) will increase coverage by 0.49%.
The diff coverage is 93.75%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1165      +/-   ##
============================================
+ Coverage     68.36%   68.85%   +0.49%     
- Complexity     1619     1627       +8     
============================================
  Files            32       32              
  Lines          3973     3994      +21     
============================================
+ Hits           2716     2750      +34     
+ Misses         1257     1244      -13     
Impacted Files Coverage Δ
controller/WebController.php 16.06% <0.00%> (-0.10%) ⬇️
controller/RestController.php 22.52% <100.00%> (+3.06%) ⬆️
model/Vocabulary.php 84.67% <100.00%> (+0.05%) ⬆️
model/VocabularyConfig.php 94.58% <100.00%> (+0.88%) ⬆️
model/sparql/GenericSparql.php 92.24% <100.00%> (+0.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d12c2d3...e7f3bef. Read the comment docs.

view/changes.twig Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented May 27, 2021

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 2 Security Hotspots
Code Smell B 20 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@joelit
Copy link
Contributor Author

joelit commented May 27, 2021

For this to work as intended, finto.css needs to be modified by removing line 480 for .nav-tabs > li.active > a :
height: 38px;

@joelit joelit added this to the 2.12 milestone Jun 2, 2021
@joelit joelit added this to Postponed in Sprint Backlog II/2021 Jun 2, 2021
@MikkoAleksanteri MikkoAleksanteri added this to Proposed product backlog items in Sprint Backlog III/2021 Sep 1, 2021
@kouralex kouralex moved this from Proposed product backlog items to Selected product backlog items in Sprint Backlog III/2021 Sep 2, 2021
@joelit joelit moved this from Selected product backlog items to In progress in Sprint Backlog III/2021 Sep 7, 2021
@joelit joelit marked this pull request as ready for review September 8, 2021 12:27
@joelit joelit moved this from In progress to Needs review in Sprint Backlog III/2021 Sep 8, 2021
WHERE {
?concept a skos:Concept ;
skos:prefLabel ?label .
FILTER (langMatches(lang(?label), '$lang'))
Copy link
Contributor

Choose a reason for hiding this comment

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

Even though it was already like this, this will filter out all concepts without a content language label, thus, resulting in incomplete listing. If one wants to fix it, one must use the fallback method found elsewhere in this file.

Copy link
Contributor

Choose a reason for hiding this comment

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

Side note: in theory, this affects also replacedByLabel.

@sonarcloud
Copy link

sonarcloud bot commented Sep 16, 2021

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 3 Security Hotspots
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

resource/css/styles.css Outdated Show resolved Hide resolved
@joelit joelit moved this from Selected product backlog items to Needs review in Sprint Backlog IV/2021 Nov 2, 2021
@sonarcloud
Copy link

sonarcloud bot commented Nov 2, 2021

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 3 Security Hotspots
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@joelit joelit merged commit b9182af into master Nov 3, 2021
Sprint Backlog II/2021 automation moved this from Postponed to Issue/PR closed Nov 3, 2021
Sprint Backlog III/2021 automation moved this from Needs review to Issue/PR closed Nov 3, 2021
Sprint Backlog IV/2021 automation moved this from Needs review to Issue/PR closed Nov 3, 2021
@joelit joelit deleted the show-newly-deprecated branch November 3, 2021 15:35
@joelit joelit moved this from Issue/PR closed to Done (verified in dev.finto.fi and documented in wiki and RN) in Sprint Backlog IV/2021 Nov 3, 2021
@osma osma added this to the 2.13 milestone Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Sprint Backlog II/2021
  
Issue/PR closed
Sprint Backlog III/2021
  
Issue/PR closed
Sprint Backlog IV/2021
  
Done (verified in dev.finto.fi and do...
Development

Successfully merging this pull request may close these issues.

None yet

3 participants