-
Notifications
You must be signed in to change notification settings - Fork 1
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
75 muokka list template taulukko muotoon #89
Conversation
Tämä on valmis mergettäväksi jo. Toimii sellaisenaan mainin kanssa, jos kohdan reference[1].format_bibtex() muuttaa muotoon reference[1].title. Tyylitiedosto styles.css on myös tärkeä tämän toimivuudelle. |
…to 75-muokka-list-template-taulukko-muotoon
Tämä myös hoitaa taskin #77, mutta täytyy odottaa, että format_bibtex metodit ovat valmiita. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #89 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 6 6
Lines 141 141
Branches 11 11
=========================================
Hits 141 141 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hyvä table. Robot-testit ovat kyllä refaktoroinnin tarpeessa, mutta hakkasin nyt toimivan ratkaisut.
Robot-testit aiheuttaa tällästä erroria:
[1206/112156.106:INFO:CONSOLE(454)] "Uncaught ReferenceError: submitted is not defined", source: http://localhost:5001/add (454)
Luulen, että tämä ongelma on kuitenkin routes/templateissa.
Lisäksi pistin ton kopiointi painikkeen piiloon, kun format_bibtex metodi ei ole vielä valmis.
Mergetään tämä kuitenkin nyt, kun toimii.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tosi hyvältä näyttää!
No description provided.