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

[MINOR] improve(docs): update the doc of the web #2907

Merged
merged 3 commits into from
Apr 23, 2024

Conversation

ch3yne
Copy link
Contributor

@ch3yne ch3yne commented Apr 12, 2024

What changes were proposed in this pull request?

Add description for creating fileset and messaging catalog, and add some tips.

Why are the changes needed?

Fix: #2100

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

N/A

@ch3yne ch3yne self-assigned this Apr 12, 2024
docs/webui.md Show resolved Hide resolved
Comment on lines +51 to +56
:::caution
If both `OAuth` and `HTTPS` are set, due to the different security permission rules of various browsers, to avoid cross-domain errors,
it is recommended to use the [Chrome](https://www.google.com/chrome/) browser for access and operation.

Such as Safari need to enable the developer menu, and select `Disable Cross-Origin Restrictions` from the develop menu.
:::
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Related issue #2100

@ch3yne ch3yne requested a review from LauraXia123 April 15, 2024 08:08
@ch3yne ch3yne force-pushed the minor-update-doc-webui-20240412 branch from 4633f1d to 02c0dc1 Compare April 23, 2024 03:58
docs/webui.md Outdated
Comment on lines 313 to 314
Design draft preview:

Copy link
Contributor

Choose a reason for hiding this comment

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

I think you don't have to mention "Design draft preview", you can delete it here and below.

docs/webui.md Outdated

Under Construction...
Preview:
Copy link
Contributor

Choose a reason for hiding this comment

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

You can remove the "Preview" here and below.

@jerryshao jerryshao added need backport Issues that need to backport to another branch branch-0.5 labels Apr 23, 2024
@jerryshao jerryshao merged commit 83fdaf9 into apache:main Apr 23, 2024
9 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 23, 2024
### What changes were proposed in this pull request?

Add description for creating fileset and messaging catalog, and add some
tips.

### Why are the changes needed?

Fix: #2100

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

N/A
diqiu50 pushed a commit to diqiu50/gravitino that referenced this pull request Jun 13, 2024
### What changes were proposed in this pull request?

Add description for creating fileset and messaging catalog, and add some
tips.

### Why are the changes needed?

Fix: apache#2100

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch-0.5 need backport Issues that need to backport to another branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug report]OAuth mode, enable https ,Safari: Network Error in login page
2 participants