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

071624.05/api revisions #8655

Merged
merged 2 commits into from
Jul 18, 2024

Conversation

chidozieononiwu
Copy link
Member

  • This adds API-Revision view showing API Revision in detail, and ability to add APIRevisions from the review page.

Copy link
Member

@praveenkuttappan praveenkuttappan left a comment

Choose a reason for hiding this comment

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

Looks fine

@praveenkuttappan
Copy link
Member

@chidozieononiwu I see an issue. I deployed this change to staging to verify the changes. I see the comments button on left side panel but click on the conversations icon is not doing anything.

Copy link
Member

@praveenkuttappan praveenkuttappan left a comment

Choose a reason for hiding this comment

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

Let's the get fix to show conversations page also along with revisions. Currently conversations icon is visible but it's not doing anything when I click on it. Also what about samples icon.

@praveenkuttappan
Copy link
Member

I have also found a bug in the revisions page when adding a new revision. Newly added revision is not shown in the revision list without refreshing the whole page. We need to refresh the revisions automatically when a new revision is available. It's not a good user experience to ask user to refresh the page to get newly added revision.

@chidozieononiwu chidozieononiwu force-pushed the 071624.05/APIRevisions branch from d9e1e15 to 43b4451 Compare July 18, 2024 20:53
@chidozieononiwu chidozieononiwu self-assigned this Jul 18, 2024
@chidozieononiwu chidozieononiwu merged commit 483f731 into Azure:main Jul 18, 2024
7 checks passed
@chidozieononiwu chidozieononiwu deleted the 071624.05/APIRevisions branch July 18, 2024 21:35
WhitShake pushed a commit to WhitShake/azure-sdk-tools that referenced this pull request Jul 24, 2024
* Add APIRevision to Review Page

* Add APIRevisions list to Review Page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants