Skip to content
This repository has been archived by the owner on May 20, 2022. It is now read-only.

update readme to add deprecation notice #388

Merged
merged 1 commit into from
May 3, 2019
Merged

update readme to add deprecation notice #388

merged 1 commit into from
May 3, 2019

Conversation

cpanato
Copy link
Contributor

@cpanato cpanato commented May 3, 2019

@cpanato cpanato requested a review from pichouk May 3, 2019 14:02
@cpanato
Copy link
Contributor Author

cpanato commented May 3, 2019

can you take a look @jasonblais thanks!

@jasonblais
Copy link
Contributor

@cpanato Is this how the texts at https://hub.docker.com/u/mattermost are generated by?

@cpanato
Copy link
Contributor Author

cpanato commented May 3, 2019

in docker hub they have a limited text field so the entire message was not possible.
the site show the readme from the repo, so this message will appears here : https://hub.docker.com/r/mattermost/mattermost-prod-app

i already can see the message which is weird but will be like that

@pichouk pichouk merged commit 65acc9a into mattermost:master May 3, 2019
@cpanato cpanato deleted the MM-15327 branch May 3, 2019 19:30
@jasonblais
Copy link
Contributor

@cpanato I was primarily looking to update the descriptions/texts that appear on the home page here https://hub.docker.com/u/mattermost

I'm a bit worried we now have a big warning at the top of the README file which makes it look like this repo is no longer maintained.

image

@jasonblais
Copy link
Contributor

@cpanato @pichouk To add, I'd propose reverting and moving this as a bullet in the "Notes" section.

If we need a shorter text for what's displayed in the description at https://hub.docker.com/u/mattermost, I can provide you one. Just let me know the approximate max character count for the description field.

pichouk added a commit that referenced this pull request May 4, 2019
@pichouk
Copy link
Contributor

pichouk commented May 4, 2019

Sorry I merged too quick. I opened a PR to revert : #389

cpanato pushed a commit that referenced this pull request May 6, 2019
chocobn69 pushed a commit to chocobn69/mattermost-docker that referenced this pull request May 7, 2019
chocobn69 pushed a commit to chocobn69/mattermost-docker that referenced this pull request May 7, 2019
nickygerritsen added a commit to Lets-Talk-NL/mattermost-docker that referenced this pull request May 28, 2019
* upstream/master: (21 commits)
  bump to 5.11.0 (mattermost#393)
  fix typo in readme (mattermost#391)
  Revert "update readme to add deprecation notice (mattermost#388)" (mattermost#389)
  include timezone for plugins that use location information. (mattermost#390)
  update readme to add deprecation notice (mattermost#388)
  update to 5.10.0
  bump to use 5.9.0
  bump to 5.8.1
  Upgrade app alpine image to 3.9
  Add Carlos Panato as maintainer
  Move Pan Luo as previous maintainer
  update MM to 5.8.0
  update MM to 5.7.2
  Enable error log to Console by default
  Upgrade to 5.7.1 (mattermost#358)
  Update docker-stack-traefik.yml
  update to 5.7.0
  bump to 5.6.3
  Add client plugins as bind mount
  add config
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants