Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃殌 Prepare v2.0.0 馃巶 #1835

Merged
merged 2 commits into from
Apr 3, 2016
Merged

馃殌 Prepare v2.0.0 馃巶 #1835

merged 2 commits into from
Apr 3, 2016

Conversation

nicosomb
Copy link
Member

@nicosomb nicosomb commented Apr 3, 2016

馃巶 馃巶 馃巶

@nicosomb nicosomb added this to the 2.0.0 milestone Apr 3, 2016
@@ -8,8 +8,7 @@ Click, save and read it when you can. It extracts content so that you can read i

More informations on our website: [wallabag.org](https://wallabag.org)

# Want to test the v2?
Keep in mind it's an **unstable** branch, everything can be broken :)
# Installa wallabag
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo here.
Do you mean "Install wallabag" or "Install a wallabag"?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

install wallabag ;-)
Thank you, will fix.

@@ -70,10 +70,6 @@

{% block footer %}{% endblock %}

<div id="warning_message">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove the associated css too ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we instead keep the div and activate the message through parameters ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(In the future, at least)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was a parameter before :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's what I thought. How come we changed it ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be moved to the config_prod.yml file and should be disabled in the config_dev.yml.
I don鈥檛 like this message in dev :)

And yep it was in the parameters.yml before, but I don鈥檛 think it should be a param that must be defined by the end user when it install wallabag.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(In the future, at least)

We'll add the <div>. I'm not fan of the "and if ever...". If we'll need it, we'll add it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don鈥檛 like this message in dev :)

Yup, it hides things, especially in mobile view. TODO for next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants