Skip to content

Commit

Permalink
Merge branch 'master' into chore/style-isolation
Browse files Browse the repository at this point in the history
  • Loading branch information
ewin committed Sep 5, 2024
2 parents 348f6d9 + a03d4b6 commit 56b80f0
Show file tree
Hide file tree
Showing 8 changed files with 358 additions and 420 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/publish-beta.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: "Publish"
name: "Publish to beta listings"
on:
push:
tags:
Expand All @@ -8,6 +8,8 @@ jobs:
build-release:
runs-on: ubuntu-latest
steps:
# TODO: somehow check if the pushed tag was a backport (i.e. it's a lower
# version than the latest beta release) and don't publish those. #964
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
- run: npm ci
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/publish-stable.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: "Publish"
name: "Publish to stable listings"
on:
push:
tags:
Expand Down
7 changes: 4 additions & 3 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,13 +102,14 @@ As an example, a release timeline might look something like this:
### Tagging a New Release

1. Make sure the working directory is clear and you're on the `master` branch.
2. Run `npm run release`. This script will prompt you for the new four-segment version number, then the release name.
2. Update `firefox_manifest.json`'s `strict_min_version` to the latest ESR version of Firefox. [Here's a link to the FF release calendar for reference](https://whattrainisitnow.com/calendar/). (TODO: automate this somehow. this is necessary because AMO will eventually prevent you from uploading extensions where this number is too low, but we want to set it to _something_ to prevent people from using the extension in browser versions where it will obviously not work.)
3. Run `npm run release`. This script will prompt you for the new four-segment version number, then the release name.
- Ensure the major.minor.patch is set correctly. You should only need to update this if the previous release was a stable release.
- If the previous release was a beta release, increment the build number by 1. If the previous release was a stable release, instead reset the build number _to_ 1.
- We generally only change the release name for major or minor bumps. If you're making a stable release and didn't change this during the beta series, make sure you update this to something appropriate for the new release. Tradition dictates it should be an adjective related to the development of the release, and an animal which start with the same letter.

The script will then automatically commit and tag the release in your local clone.
3. Verify that the commit created by the release script contains nothing except changes to the version strings in the manifest files.
4. Push the commit and tag: `git push && git push --tags`.
4. Verify that the commit created by the release script contains nothing except changes to the version strings in the manifest files.
5. Push the commit and tag: `git push && git push --tags`.

Once your tag is pushed to Github, the CI pipeline will generate release builds and automatically submit them to extension stores. Beta releases will be sent only to the beta listings; stable releases will result in updates to both the stable _and_ beta listings, with the beta listing receiving a beta-flagged build containing otherwise the same code.
2 changes: 1 addition & 1 deletion extension/chrome_manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"author": "toolbox team",
"short_name": "toolbox",
"description": "A set of tools to be used by moderators on reddit in order to make their jobs easier.",
"version": "7.0.0.3",
"version": "7.0.0.6",
"version_name": "7.0.0: \"Oh God Erin What Are You Doing\"",
"incognito": "split",
"permissions": [
Expand Down
18 changes: 13 additions & 5 deletions extension/data/modules/macros.js
Original file line number Diff line number Diff line change
Expand Up @@ -175,11 +175,19 @@ export default new Module({
getConfig(info.subreddit, (success, config) => {
// if we're a mod, add macros to top level reply button.
if (success && config.length > 0) {
const macroButtonHtml =
`<select class="tb-macro-select tb-action-button" data-subreddit="${info.subreddit}"><option value=${MACROS}>macros</option></select>`;
$body.find('.ThreadViewerReplyForm__replyFooter .selectWrapper').before(
`<div class="tb-usertext-buttons tb-macro-newmm">${macroButtonHtml}</div>`,
);
const macroButtonHtml = `
<select class="tb-macro-select tb-action-button" data-subreddit="${info.subreddit}">
<option value=${MACROS}>macros</option>
</select>
`;
$body.find(`
:is(
.ThreadViewerReplyForm__replyFooter,
.ThreadViewerReplyForm__replyFooterGroup
) .selectWrapper
`).before(`
<div class="tb-usertext-buttons tb-macro-newmm">${macroButtonHtml}</div>
`);

populateSelect('.tb-macro-select', info.subreddit, config, 'modmail');
}
Expand Down
4 changes: 2 additions & 2 deletions extension/firefox_manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@
"author": "toolbox team",
"short_name": "toolbox",
"description": "A set of tools to be used by moderators on reddit in order to make their jobs easier.",
"version": "7.0.0.3",
"version": "7.0.0.6",
"version_name": "7.0.0: \"Oh God Erin What Are You Doing\"",
"browser_specific_settings": {
"gecko": {
"id": "yes@jetpack",
"strict_min_version": "57.0"
"strict_min_version": "115.0"
}
},
"permissions": [
Expand Down
Loading

0 comments on commit 56b80f0

Please sign in to comment.