Rails8#399
Closed
echarp wants to merge 96 commits into
Closed
Conversation
Dépendant de la gemme rails-sass-images qui n'est plus développée
l'évolution de la gemme
Ajout de "optional: true" pour les modèles qui permettent une valeur "null". Document du comportement belongs_to: https://sipsandbits.com/2015/09/21/whats-new-in-rails-5/#belongs_toisrequiredbydefault
Il y avait deux classes pour dynamiquement chercher des templates entre les espaces de rédaction et modération, après tests et recherches il semble que ça ne soit pas (ou plus?) nécessaire.
les espaces de rédaction et modération
Tests et validations nécessaires, mais en pause car migration vers 7 prévue
…s non fonctionnelles
La méthode est maintenant nommée File.exist?
qui sont à peu près touts couverts
(en tous cas du code testé d'après la gemme simplecov...)
(semble non utilisé dans l'IHM)
Il pourrait y avoir d'autres cas, cela vient de la simplification de recherche des chemins de partiaux, suite (ou grâce) à la migration à rails 7
Il manque les contrôleurs d'API dans cette couverture...
- Upgraded Rails from 7.1 to 7.2 in Gemfile and related files. - Updated config/initializers/secrets.rb to load secrets from environment variables if present, falling back to credentials if not. - Ensured that `secret_key_base`, `push_secret`, and `devise_pepper` are loaded securely. - Updated config/secrets.yml.sample to reflect the changes.
The model specifies a max length of 160 for the title column, but the
form template specified a max length of 100 for the title field.
Indeed, if you look in db/schema.rb you'll find:
t.string "title", limit: 160, null: false
- Target Debian Trixie (stable) - Update MySQL/MariaDB package names - Simplify Bundler installation (now included with Ruby) - Improve document structure and add syntax highlighting
12 tasks
ensure markdown export is using only LF as end of line
Make the bookmark title field length consistent with the model
Make the news and comment title field length consistent with the model
Member
|
J'ai fusionné à la main la branch echarp:rails8 dans linuxfrorg:rails7 parce que j'ai cassé en début de semaine le workflow prévu par Github (j'avais fait un rebase de linuxfrorg:rails7 sur linuxfrorg:master pour simplifier cette fusion, mais ça a empiré la situation). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.