Merge pull request #1588 from Budibase/fix/no-page-load

Fix/no page load
This commit is contained in:
Michael Drury 2021-05-28 20:03:57 +01:00 committed by GitHub
commit 742f7d6dc1
4 changed files with 1310 additions and 19 deletions

View File

@ -291,7 +291,7 @@
/> />
{#if relationshipOptions && relationshipOptions.length > 0} {#if relationshipOptions && relationshipOptions.length > 0}
<RadioGroup <RadioGroup
disabled={originalName} disabled={originalName != null}
label="Define the relationship" label="Define the relationship"
bind:value={field.relationshipType} bind:value={field.relationshipType}
options={relationshipOptions} options={relationshipOptions}

View File

@ -9,7 +9,8 @@ function getAppRole(appId, user) {
if (!user.roles) { if (!user.roles) {
return user return user
} }
user.roleId = user.roles[appId] // always use the deployed app
user.roleId = user.roles[getDeployedAppID(appId)]
if (!user.roleId) { if (!user.roleId) {
user.roleId = BUILTIN_ROLE_IDS.PUBLIC user.roleId = BUILTIN_ROLE_IDS.PUBLIC
} }
@ -97,8 +98,6 @@ exports.deleteGlobalUser = async (ctx, globalId) => {
} }
exports.getGlobalUsers = async (ctx, appId = null, globalId = null) => { exports.getGlobalUsers = async (ctx, appId = null, globalId = null) => {
// always use the deployed app
appId = getDeployedAppID(appId)
const endpoint = globalId const endpoint = globalId
? `/api/admin/users/${globalId}` ? `/api/admin/users/${globalId}`
: `/api/admin/users` : `/api/admin/users`
@ -136,7 +135,6 @@ exports.getGlobalSelf = async (ctx, appId = null) => {
} }
exports.addAppRoleToUser = async (ctx, appId, roleId, userId = null) => { exports.addAppRoleToUser = async (ctx, appId, roleId, userId = null) => {
appId = getDeployedAppID(appId)
let user, let user,
endpoint, endpoint,
body = {} body = {}

File diff suppressed because it is too large Load Diff

View File

@ -77,6 +77,9 @@ const lengthConstraint = maxLength => value => {
} }
const numericalConstraint = (constraint, error) => value => { const numericalConstraint = (constraint, error) => value => {
if (value == null || value === "") {
return null
}
if (isNaN(value)) { if (isNaN(value)) {
return "Must be a number" return "Must be a number"
} }