Render changelog.md instead of opening a webview #10

Closed
Ghost wants to merge 6 commits from (deleted):changelog_screen into master
First-time contributor

J'ai essayé de faire un truc un peu plus propre cette fois-ci sachant que j'ai ajouté une nouvelle dépendance: react-native-markdown-display

J'ai essayé de faire un truc un peu plus propre cette fois-ci sachant que j'ai ajouté une nouvelle dépendance: react-native-markdown-display
vergnet self-assigned this 2020-09-03 11:39:21 +02:00
vergnet approved these changes 2020-09-03 12:50:34 +02:00
@ -0,0 +1,47 @@
import React from 'react';
import Markdown from "react-native-markdown-display";
import {withTheme} from "react-native-paper";
/**
Owner

Tu n'as pas défini les PropsType à utiliser pour appeler ce composant (regarde un autre composant que j'ai fait pour comparer)

Tu n'as pas défini les PropsType à utiliser pour appeler ce composant (regarde un autre composant que j'ai fait pour comparer)
@ -0,0 +12,4 @@
/>
*/
class TextFileReader extends React.Component {
constructor(props) {
Owner

et donc ici tu dois dire que props est de type PropsType (props: PropsType)

et donc ici tu dois dire que props est de type PropsType (props: PropsType)
@ -0,0 +20,4 @@
}
componentDidMount() {
this.readTextFile(this.props.text);
Owner

Si tu as bien configuré eslint, regarde bien son erreur.

Il faut faire const text = {this.props}, c'est ce qui s'appelle destructuring

Si tu as bien configuré eslint, regarde bien son erreur. Il faut faire const text = {this.props}, c'est ce qui s'appelle destructuring
@ -0,0 +24,4 @@
}
readTextFile = file => {
let xhr = new XMLHttpRequest();
Owner

Utilise pas XMLHttpRequest pour faire des requêtes web c'est super chiant à utiliser. à la place, utilise la fonction fetch. Tu peux aller voir dans src/utils/WebData.js pour voir comment faire

Utilise pas XMLHttpRequest pour faire des requêtes web c'est super chiant à utiliser. à la place, utilise la fonction fetch. Tu peux aller voir dans src/utils/WebData.js pour voir comment faire
@ -0,0 +37,4 @@
xhr.send(null);
};
render(){
Owner

Eslint va t'embêter avec le type de retour, il suffit d'écrire render: React.Node () { pour qu'il arrête de râler

Eslint va t'embêter avec le type de retour, il suffit d'écrire `render: React.Node () {` pour qu'il arrête de râler
@ -0,0 +5,4 @@
import CollapsibleScrollView from '../../components/Collapsible/CollapsibleScrollView';
import TextFileReader from "../../components/Screens/MarkdownRenderScreen";
const url = "https://git.etud.insa-toulouse.fr/leban/application-amicale/raw/branch/master/Changelog.md";
Owner

Mauvais url, utilise celui du master du dépôt officiel

Mauvais url, utilise celui du master du dépôt officiel
Ghost closed this pull request 2020-09-05 01:51:07 +02:00

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: vergnet/application-amicale#10
No description provided.