Skip to content
This repository has been archived by the owner on Jun 15, 2021. It is now read-only.

Add proposals.adoc #40

Merged
merged 3 commits into from
Apr 23, 2018
Merged

Add proposals.adoc #40

merged 3 commits into from
Apr 23, 2018

Conversation

cbeams
Copy link
Contributor

@cbeams cbeams commented Apr 23, 2018

This doc captures the clarifications and refinements to the proposals
process proposed in bisq-network/proposals#11.

This doc captures the clarifications and refinements to the proposals
process proposed in bisq-network/proposals#11.
Copy link

@mrosseel mrosseel left a comment

Choose a reason for hiding this comment

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

ACK

@cbeams cbeams self-assigned this Apr 23, 2018

Once a proposal is submitted, a two-week review period follows. During this period, interested reviewers should read, discuss and ultimately react to the proposal as follows:

- 👍: I agree with the proposal and want to see it enacted
Copy link

Choose a reason for hiding this comment

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

This renders as square. I think something else was intended here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Interesting, looking into it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is what I see in the deploy preview at https://deploy-preview-40--bisq-docs.netlify.com/proposals.html:

image

Can you share a screenshot of what you see?

Copy link

Choose a reason for hiding this comment

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

proposals

Choose a reason for hiding this comment

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

screen shot 2018-04-23 at 15 03 28

Copy link

Choose a reason for hiding this comment

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

I guess I miss some font.

cbeams added 2 commits April 23, 2018 15:21
In response to review feedback.
This reverts commit 0ba1084, because it
made no difference on the affected OS (Gentoo Linux). It seems the only
way to guarantee that these emoji show up properly everywhere is to use
images instead of characters. This is what GitHub does, for example,
everywhere they use these symbols.
@cbeams cbeams merged commit f051a20 into bisq-network:master Apr 23, 2018
cbeams added a commit that referenced this pull request Apr 23, 2018
cbeams added a commit to bisq-network/proposals that referenced this pull request Apr 23, 2018
cbeams added a commit to bisq-network/proposals that referenced this pull request Apr 23, 2018
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