omorest / weatherweb-front

Hi there 👋

• 💻 Computer Science Engineer
• 💘 Passionate about learning good practices, architectures, testing, etc.
• 🌱 Learning something every day.

Contact with me 🙋🏽‍♂️

---

weatherweb-front's Issues

Error Handler

Eslint

Method to parse the most interesting data from weather

Style of web

Webpack configuration

Method to get all data weather from open weather api

Github action

Parsers

https://github.com/omorest/WeatherWeb/blob/f00f1221a785ae65a0836fddf0af761c4cd541fc/src/index.js#L13

@omorest Holaa bro! 🙌 he estado mirándote el código por lo que habíamos hablado, y justo en desde esta línea veo que haces lo que se suele llamar parsear los datos de la request. En vez de hacerlo aquí puedes crearte una función toWeatherInformation o similar y meterle ahí toda las transformación que haces.

Esa función la llamas luego antes del return en el request.js tal que así:

export const getDataWeatherByLatLon = async (latitude, longitude) => {
	const url = `https://api.openweathermap.org/data/2.5/weather?lat=${latitude}&lon=${longitude}&units=metric&appid=${process.env.KEY}`;
	const res = await fetch(url);
	const data = await res.json();
	return toWeatherInformation(data);
}
Puedes crear un fichero dentro de un directorio weather y meterle ahí el parsers.js de esta manera puedes tener algo así como: 

src/api/weather/requests.js ó endpoints.js
                         /parsers.js
                         /models.js // esto no pero por ejemplo si usas typescript siempre los tipos van en otro directorio aparte y así lo tienes todo organizado. 

De esta manera en la función de initInformationWeather te quedaría algo así:

const iniInformationWeather = () => {
	async function success(position) {
		const {latitude, longitude} = position.coords;
		const dataWeather = await getDataWeatherByLatLon(latitude, longitude);
		createInformationWeather(weatherInfoRelevant);
	};
	
	navigator.geolocation.getCurrentPosition(success);
}

La idea es que quitas de la función initInformationWeather la complejidad del parseo ya que su responsabilidad no es hacerlo, sino tal y como lo haces, coger la posición, llamar a la request y crear la información. Los parseos y cambios de modelo desde el back al front y viceversa se suelen hacer siempre en el punto más cercano a la salida o entrada de datos del front, en nuestro caso siempre serán las request. Por ello los parsers siempre se suelen poner ahí 😄

Espero haberte ayudado bro! 🙌

Inputs to get information from another places

Current location with geolocation

README

editorConfig

Deployment on Vercel