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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: settings use value instead of placeholder #36

Merged
merged 1 commit into from
Aug 21, 2020

Conversation

thomasguillot
Copy link
Contributor

All Submissions:

Changes proposed in this Pull Request:

Instead of using input's placeholder for the default content, I suggest we display it as a value which will be easier for a user to tweak if needed.

How to test the changes in this Pull Request:

  1. Go to Sponsors > Settings. Notice the default settings are a placeholder.
  2. Switch to this branch. Now the default settings are fully displayed.
  3. Add new sponsor and publish, check the post if it's displayed correctly.
  4. Edit the settings (via the Editor AND globally via Settings)
  5. Check the post

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

Copy link
Contributor

@dkoo dkoo left a comment

Choose a reason for hiding this comment

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

Great idea, thanks for implementing!

@thomasguillot thomasguillot merged commit 7276c5d into master Aug 21, 2020
@thomasguillot thomasguillot deleted the update/settings-value branch August 21, 2020 17:16
matticbot pushed a commit that referenced this pull request Aug 25, 2020
# [1.2.0-alpha.1](v1.1.0...v1.2.0-alpha.1) (2020-08-25)

### Bug Fixes

* a safer way of overwriting the prop ([#39](#39)) ([c86e555](c86e555))
* checkout --> checkout_with_workspace ([#34](#34)) ([8138a11](8138a11))
* install rsync as part of release job ([efa53fc](efa53fc))
* merge conflicts ([9a2ed5d](9a2ed5d))
* semantic release config ([2366839](2366839))
* semantic release config ([#29](#29)) ([e116c58](e116c58))

### Features

* settings use value instead of placeholder ([#36](#36)) ([7276c5d](7276c5d))
* update sponsors icon to Material Design ([#35](#35)) ([b2719bf](b2719bf))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 1.2.0-alpha.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

matticbot pushed a commit that referenced this pull request Aug 25, 2020
# [1.2.0](v1.1.0...v1.2.0) (2020-08-25)

### Bug Fixes

* a safer way of overwriting the prop ([#39](#39)) ([c86e555](c86e555))
* checkout --> checkout_with_workspace ([#34](#34)) ([8138a11](8138a11))
* install rsync as part of release job ([efa53fc](efa53fc))
* merge conflicts ([9a2ed5d](9a2ed5d))
* semantic release config ([2366839](2366839))
* semantic release config ([#29](#29)) ([e116c58](e116c58))

### Features

* settings use value instead of placeholder ([#36](#36)) ([7276c5d](7276c5d))
* update sponsors icon to Material Design ([#35](#35)) ([b2719bf](b2719bf))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 1.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

3 participants