Rails8 #399
Closed
Rails8 #399
Conversation
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
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...
This commit introduces two new global constants MY_PUBLIC_URL and IMG_PUBLIC_URL which should be used now to build URLs. For alpha and production Ruby environments, they are fixed with their real values directly in the `config/environments/*.rb` files. For the development Ruby environment, they are defined by default to `http://dlfp.lo` and `http://image.dlfp.lo`. You can customize either by updating the `config/environments/development.rb` file or by setting the *process* environment variables `DOMAIN`, `DOMAIN_HTTP_PORT`, `IMAGE_DOMAIN` and `IMAGE_DOMAIN_HTTP_PORT`. For containers, you can set these variables in the `deployment/default.env` file.
- 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). |
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.