Show warning when docker image ends with ":latest"

[jennydaman]

It is bad practice to use the :latest docker images, because the app (e.g. its command-line options) are subject to change in later versions.

The warning message might briefly explain why it's a bad practice, and suggest a remedy:

docker tag user/pl-mything:latest user/pl-mything:1.0.1
docker push user/pl-mything:1.0.1

[DivyanshiSingh]

Hey, @mairin I would like to work on this issue. Please assign me this.

[DivyanshiSingh]

Hey, @jennydaman can you provide some more information that will help me in solving this issue.
I am not getting where to write the script for testing the build command.

[jennydaman]

script for testing the build command

I am not sure what you mean by this.

Here's a hacked mock-up

Here the user is trying to create a plugin. They've specified the "Docker Image" (dock_image) as fnndsc/pl-surf. This is a bad practice so we show the warning message

Please tag your Docker image by version.

In that box, you could include a link to a help article explaining why and how, or design a layout to provide immediate help. e.g. show a code block

docker tag fnndsc/pl-surf fnndsc/pl-surf:1.0.1
docker push fnndsc/pl-surf:1.0.1

If the user's input for dock_image ends with :latest instead, then show the warning

The :latest tag is discouraged. Please tag your Docker image by version.

Input validation code is found here

ChRIS_store_ui/src/components/CreatePlugin/CreatePlugin.js

Lines 223 to 242 in 0db5186

	// If all the fields are empty in submission
	if (errors.length === 3 && missingRepresentationString !== '') {
	return this.handleError('All fields are required.');
	// If one fields is empty
	} else if (errors.length === 1) {
	return this.handleError(`Please ${missingRepresentationString}enter
	the ${errors[0]}`);
	// If two fields are empty
	} else if (errors.length === 2) {
	return this.handleError(`Please ${missingRepresentationString}enter
	the ${errors[0]} and ${errors[1]}`);
	// If more than Plugin Representation or two fields are empty or
	}
	// If only the Plugin Representation is missing
	if (errors.length === 0) {
	return this.handleError('Please upload the plugin representation');
	}
	const lastError = errors.pop();
	return this.handleError(`Please ${missingRepresentationString}enter
	the ${errors.join(', ')}, and ${lastError}`);

[DivyanshiSingh]

Ok, I got it @jennydaman.
I was moving in the wrong direction...This is a great help. Thank you