Skip to content

Commit

Permalink
Merge branch 'main' into marcaaron-addOnfidoSDKs
Browse files Browse the repository at this point in the history
  • Loading branch information
tgolen committed Jun 4, 2021
2 parents 215f1e4 + 2345457 commit 6cf1a56
Show file tree
Hide file tree
Showing 31 changed files with 243 additions and 76 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ const DEFAULT_PAYLOAD = {
};

const pullRequestNumber = ActionUtils.getJSONInput('PULL_REQUEST_NUMBER', {required: false}, null);
const user = ActionUtils.getJSONInput('USER', {required: false}, null);
const titleRegex = ActionUtils.getJSONInput('TITLE_REGEX', {required: false}, null);
const user = core.getInput('USER', {required: false});
let titleRegex = core.getInput('TITLE_REGEX', {required: false});

if (pullRequestNumber) {
console.log(`Looking for pull request w/ number: ${pullRequestNumber}`);
Expand All @@ -21,6 +21,7 @@ if (user) {
}

if (titleRegex) {
titleRegex = new RegExp(titleRegex);
console.log(`Looking for pull request w/ title matching: ${titleRegex.toString()}`);
}

Expand Down Expand Up @@ -58,7 +59,10 @@ if (pullRequestNumber) {
.then(({data}) => outputMergeCommitHash(data))
.catch(handleUnknownError);
} else {
GithubUtils.octokit.pulls.list(DEFAULT_PAYLOAD)
GithubUtils.octokit.pulls.list({
...DEFAULT_PAYLOAD,
state: 'all',
})
.then(({data}) => {
const matchingPR = _.find(data, PR => PR.user.login === user && titleRegex.test(PR.title));
outputMergeCommitHash(matchingPR);
Expand Down
10 changes: 7 additions & 3 deletions .github/actions/getMergeCommitForPullRequest/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ const DEFAULT_PAYLOAD = {
};

const pullRequestNumber = ActionUtils.getJSONInput('PULL_REQUEST_NUMBER', {required: false}, null);
const user = ActionUtils.getJSONInput('USER', {required: false}, null);
const titleRegex = ActionUtils.getJSONInput('TITLE_REGEX', {required: false}, null);
const user = core.getInput('USER', {required: false});
let titleRegex = core.getInput('TITLE_REGEX', {required: false});

if (pullRequestNumber) {
console.log(`Looking for pull request w/ number: ${pullRequestNumber}`);
Expand All @@ -31,6 +31,7 @@ if (user) {
}

if (titleRegex) {
titleRegex = new RegExp(titleRegex);
console.log(`Looking for pull request w/ title matching: ${titleRegex.toString()}`);
}

Expand Down Expand Up @@ -68,7 +69,10 @@ if (pullRequestNumber) {
.then(({data}) => outputMergeCommitHash(data))
.catch(handleUnknownError);
} else {
GithubUtils.octokit.pulls.list(DEFAULT_PAYLOAD)
GithubUtils.octokit.pulls.list({
...DEFAULT_PAYLOAD,
state: 'all',
})
.then(({data}) => {
const matchingPR = _.find(data, PR => PR.user.login === user && titleRegex.test(PR.title));
outputMergeCommitHash(matchingPR);
Expand Down
7 changes: 5 additions & 2 deletions .github/workflows/cherryPick.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ jobs:

- name: Create branch for new pull request
run: |
git config user.name ${{ github.actor }}
git checkout -b cherry-pick-staging-${{ github.event.inputs.PULL_REQUEST_NUMBER }}
git push --set-upstream origin cherry-pick-staging-${{ github.event.inputs.PULL_REQUEST_NUMBER }}
Expand Down Expand Up @@ -99,9 +100,11 @@ jobs:
USER: OSBotify
TITLE_REGEX: Update version to ${{ env.NEW_VERSION }}

- name: Cherry-pick the merge commit to new branch
- name: Cherry-pick the merge commits to new branch
id: cherryPick
run: git cherry-pick ${{ steps.getCPMergeCommit.outputs.MERGE_COMMIT_SHA }} ${{ steps.getVersionBumpMergeCommit.MERGE_COMMIT_SHA }} --mainline 1
run: |
git fetch
git cherry-pick ${{ steps.getCPMergeCommit.outputs.MERGE_COMMIT_SHA }} ${{ steps.getVersionBumpMergeCommit.outputs.MERGE_COMMIT_SHA }} --mainline 1
continue-on-error: true

# If there is a merge conflict, we'll just commit what we have,
Expand Down
4 changes: 2 additions & 2 deletions android/app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -148,8 +148,8 @@ android {
minSdkVersion rootProject.ext.minSdkVersion
targetSdkVersion rootProject.ext.targetSdkVersion
multiDexEnabled rootProject.ext.multiDexEnabled
versionCode 1001006100
versionName "1.0.61-0"
versionCode 1001006300
versionName "1.0.63-0"
}
splits {
abi {
Expand Down
3 changes: 1 addition & 2 deletions desktop/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,6 @@ const mainWindow = (() => {
width: 1200,
height: 900,
webPreferences: {
enableRemoteModule: true,
nodeIntegration: true,
},
titleBarStyle: 'hidden',
Expand Down Expand Up @@ -229,7 +228,7 @@ const mainWindow = (() => {
ipcMain.on(ELECTRON_EVENTS.REQUEST_VISIBILITY, (event) => {
// This is how synchronous messages work in Electron
// eslint-disable-next-line no-param-reassign
event.returnValue = browserWindow.isFocused();
event.returnValue = browserWindow && browserWindow.isFocused();
});

// This allows the renderer process to bring the app
Expand Down
4 changes: 2 additions & 2 deletions ios/ExpensifyCash/Info.plist
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
<key>CFBundlePackageType</key>
<string>APPL</string>
<key>CFBundleShortVersionString</key>
<string>1.0.61</string>
<string>1.0.63</string>
<key>CFBundleSignature</key>
<string>????</string>
<key>CFBundleURLTypes</key>
Expand All @@ -30,7 +30,7 @@
</dict>
</array>
<key>CFBundleVersion</key>
<string>1.0.61.0</string>
<string>1.0.63.0</string>
<key>ITSAppUsesNonExemptEncryption</key>
<false/>
<key>LSApplicationQueriesSchemes</key>
Expand Down
4 changes: 2 additions & 2 deletions ios/ExpensifyCashTests/Info.plist
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,10 @@
<key>CFBundlePackageType</key>
<string>BNDL</string>
<key>CFBundleShortVersionString</key>
<string>1.0.61</string>
<string>1.0.63</string>
<key>CFBundleSignature</key>
<string>????</string>
<key>CFBundleVersion</key>
<string>1.0.61.0</string>
<string>1.0.63.0</string>
</dict>
</plist>
68 changes: 37 additions & 31 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "expensify.cash",
"version": "1.0.61-0",
"version": "1.0.63-0",
"author": "Expensify, Inc.",
"homepage": "https://expensify.cash",
"description": "Expensify.cash is the next generation of Expensify: a reimagination of payments based atop a foundation of chat.",
Expand Down Expand Up @@ -133,7 +133,7 @@
"css-loader": "^5.2.4",
"detox": "^17.8.3",
"diff-so-fancy": "^1.3.0",
"electron": "^9.2.0",
"electron": "^11.4.8",
"electron-builder": "^22.8.0",
"electron-notarize": "^1.0.0",
"electron-reloader": "^1.2.0",
Expand Down
2 changes: 2 additions & 0 deletions src/CONST.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ const CONST = {
ALL: 'all',
CHRONOS_IN_CASH: 'chronosInCash',
IOU: 'IOU',
PAY_WITH_EXPENSIFY: 'payWithExpensify',
},
BUTTON_STATES: {
DEFAULT: 'default',
Expand Down Expand Up @@ -211,6 +212,7 @@ const CONST = {
// not be changed.
PAYMENT_TYPE: {
ELSEWHERE: 'Elsewhere',
EXPENSIFY: 'Expensify',
PAYPAL_ME: 'PayPal.me',
VENMO: 'Venmo',
},
Expand Down
1 change: 1 addition & 0 deletions src/ONYXKEYS.js
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ export default {
REPORT_ACTIONS_DRAFTS: 'reportActionsDrafts_',
REPORT_USER_IS_TYPING: 'reportUserIsTyping_',
REPORT_IOUS: 'reportIOUs_',
POLICY: 'policy_',
},

// Indicates which locale should be used
Expand Down
2 changes: 2 additions & 0 deletions src/components/FAB/FAB.js
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,8 @@ class FAB extends PureComponent {

return (
<AnimatedPressable
accessibilityLabel={this.props.accessibilityLabel}
accessibilityRole={this.props.accessibilityRole}
onPress={this.props.onPress}
style={[
styles.floatingActionButton,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,15 +1,12 @@
import {Animated} from 'react-native';
import PropTypes from 'prop-types';
import {windowDimensionsPropTypes} from '../../withWindowDimensions';

const propTypes = {
/** GrowlNotification content */
children: PropTypes.node.isRequired,

/** GrowlNotification Y postion, required to show or hide with fling animation */
translateY: PropTypes.instanceOf(Animated.Value).isRequired,

...windowDimensionsPropTypes,
};

export default propTypes;
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@
import React from 'react';
import {Animated} from 'react-native';
import styles from '../../../styles/styles';
import withWindowDimensions from '../../withWindowDimensions';
import propTypes from './GrowlNotificationContainerPropTypes';
import withWindowDimensions, {windowDimensionsPropTypes} from '../../withWindowDimensions';
import GrowlNotificationContainerPropTypes from './GrowlNotificationContainerPropTypes';

const propTypes = {
...GrowlNotificationContainerPropTypes,
...windowDimensionsPropTypes,
};

const GrowlNotificationContainer = ({children, translateY, isSmallScreenWidth}) => (
<Animated.View
Expand All @@ -18,5 +23,6 @@ const GrowlNotificationContainer = ({children, translateY, isSmallScreenWidth})
);

GrowlNotificationContainer.propTypes = propTypes;
GrowlNotificationContainer.displayName = 'GrowlNotificationContainer';

export default withWindowDimensions(GrowlNotificationContainer);
Loading

0 comments on commit 6cf1a56

Please sign in to comment.