Skip to content

Conversation

@yang-chengg
Copy link
Owner

No description provided.

Copy link

@mparkhe mparkhe left a comment

Choose a reason for hiding this comment

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

This looks good. Check the path in the deploy script

}

# Validate the current folder
[[ -d "./src" && -f "./src/app.yaml" ]] || { echo_red "Error: Couldn't find app.yaml. \nPlease run this script from the //feature-store/registry directory."; exit 1; }
Copy link

Choose a reason for hiding this comment

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

Check the path


def get_user_access_token():
if "user_access_token" not in st.session_state:
st.session_state.user_access_token = st.context.headers.get('x-forwarded-access-token')
Copy link

Choose a reason for hiding this comment

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

please add a comment to what is happening here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants