| Commit message (Collapse) | Author | Age | Files | Lines | 
| | 
| 
| 
| 
| 
| 
| 
|  | 
It makes it easier to change a route if needed and it avoid typo
mistakes.
I also refactored a bit the concerned files to be complient with the
new ESlint config. However, I should rewrite the pages to reduce
the number of statements.
 | 
| | 
| 
| 
| 
| 
|  | 
Next expect a default export for pages so only those components should
use default exports. Everything else should use named exports to
reduce the number of import statements.
 | 
| | 
| 
| 
| 
| 
|  | 
Using paths aliases starting with "@" can be confusing and can lead to
conflict with existings modules. I prefer to use relative paths to
avoid extra configuration in tools because of these aliases.
 | 
| |  | 
 | 
| |  | 
 | 
| | 
| 
| 
| 
| 
|  | 
It will be useful with Docker. Instead of cloning the project with a
different configuration, I can manage two different configuration
thanks to dotenv and some checking inside the app.
 | 
| | 
| 
| 
| 
|  | 
I also renamed and changed the format of some environment variables so
I can reuse them inside the CSP security header.
 | 
| |  | 
 | 
| | 
| 
| 
| 
| 
|  | 
I cannot use formatjs to translate the website baseline since I need
an async function to load the message. If I use the getIntlInstance
helper outside NextJS, webpack is complaining about fs and path.
 | 
| |  | 
 | 
| | 
| 
| 
| 
|  | 
I will need the country code, so I think it makes more sense to gather
them inside the same property.
 | 
| |  | 
 | 
| |  | 
 | 
|   
  
  
  
  
   | 
I could loop until hasNextPage is false, but I'm not sure if it is the
best solution. So, for now, only the first 100 posts will be added to
the file.
 |