mattermost-community/mattermost-plugin-memes

Modular library

Open

#47 opened on Mar 26, 2021

View on GitHub
 (1 comment) (4 reactions) (0 assignees)Go (33 forks)auto 404
Difficulty/2:MediumHacktoberfestHelp WantedTech/GoType/EnhancementUp For Grabs

Repository metrics

Stars
 (80 stars)
PR merge metrics
 (PR metrics pending)

Description

Hello,

Would it be possible to convert the hard coded list of images and slots into a modular one ? A folder with

  • The images
  • A yaml file for each image, with the same name, containing slots (as actually), And a script that build automatically the list with what it find in the library. (I think it is the file "assets.go" but I'm not sure)

According to this PR: https://github.com/mattermost/mattermost-plugin-memes/pull/46/files the difficult thing seems to be the line of bytes, is it the file hard coded ?

It can be manually launched (no need to rebuild every morning) and it could return errors in command line, as it would be only done by the admin and not the users.

(sorry I'm no dev)

In any case, thank you for the plugin so far !

Contributor guide

Research direction
Inspect the current hard coded list in assets.go and refer to PR #46 to understand the structure. Design a modular system with a folder containing images and YAML files defining slots. Implement a script to build the list dynamically from the folder contents.
Tech stack
go
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
GoYAML
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60