Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Edit issue title #2618

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BrianLitwin
Copy link
Member

@BrianLitwin BrianLitwin commented Jan 20, 2019

This has three pieces of key functionality:

  • sends a v3 network request to update the issue title
  • updates the IssueTitleSectionController (the title the user sees
    when they open an Issue)
  • updates the Issue Timeline

These can be tested/verified manually. I also added a unit test
to verify that the User's interaction with the EditIssueTitleViewController
works as expected, eg a network request won't be sent unless a user
edits the issue title and calls onSave().

This is an update of #2348 - in this version of the PR,
EditIssueTitle's behavior is intended to be very similar to
Labels, People, and Milestones. Ie it updates the UI optimistically and
these updates are triggered by ContextMenuWillDismiss().

I didn't add any functionality to update Bookmarks after an
issue title is edited because my understanding is that the
bookmark titles are not persisted locally anymore but are retrieved from
the github api.

I updated some string constants (requested in original pr) and I removed the
width parameter from a helper function titleStringSizing because
it was convenient and I couldn't find anywhere that parameter was used

This has three pieces of key functionality:
* sends a v3 network request to update the issue title
* updates the IssueTitleSectionController (the title the user sees
when they open an Issue)
* updates the Issue Timeline

These can be tested/verified manually. I also added a unit test
to verify that the User's interaction with the EditIssueTitleViewController
works as expected, eg a network request won't be sent unless a user
edits the issue title and calls onSave().

This is an update of GitHawkApp#2348 - in this version of the PR,
 EditIssueTitle's behavior is intended to be very similar to
 Labels, People, and Milestones. Ie it updates the UI optimistically and
 these updates are triggered by ContextMenuWillDismiss().

I didn't add any funcionality to update Bookmarks after an
issue title is edited because my understanding is that the
bookmark titles are not stored anymore but are retrieved from
 the github api.

I updated some string constants (requested in original pr) and I removed the
 width parameter from a helper function [titleStringSizing] because
it was convenient and I couldn't find anywhere that parameter was used

[titleStringSizing]: https:/GitHawkApp/GitHawk/blob/45e9cf6428d5edfe00a73eb18450c50c03871510/Classes/Issues/IssueViewModels.swift#L13
@BrianLitwin
Copy link
Member Author

BrianLitwin commented Jan 20, 2019

Just a note @rnystrom @Sherlouk @Huddie or anybody who can help: I have NO IDEA what's going on with the noise in .pbxproj on this pr ... All I know is that it was very difficult to add a new file to 'local pods`. Lmk if y'all have any solutions or ideas.

@Huddie Huddie added the 💤 awaiting review Pull Request is awaiting code reviews label Jan 21, 2019
@Huddie
Copy link
Collaborator

Huddie commented Mar 11, 2019

@BrianLitwin still on this? I can take a look @ .pbxproj if you are. See if I can think of something.
Landing this would be sweet

Sent with GitHawk

@BrianLitwin
Copy link
Member Author

BrianLitwin commented Mar 11, 2019

@Huddie The feature works fine locally but I haven't been able to add files to the GitHub pod without a ton of noise in my PR. If you wanna check it out and see if you can fix, that'd be great.

@Huddie
Copy link
Collaborator

Huddie commented Mar 11, 2019

@BrianLitwin I think the files are fine here. Seems like a lot of random stuff but I'm getting the same thing when I try it out. I removed the commit and re-ran the project and the same proj file is getting generated. @BasThomas want to take a quick look?
@BrianLitwin Maybe just resolve the conflict and we can merge it in. Works for me on my local and if Bas takes a look it may be good to go.

@BasThomas
Copy link
Collaborator

Would be awesome to get this in indeed. I’ll take another look when I get the time!

Sent with GitHawk

@BasThomas
Copy link
Collaborator

🤔 why did you close this one @BrianLitwin?

@BrianLitwin
Copy link
Member Author

@BasThomas on accident lol

@BrianLitwin BrianLitwin reopened this Mar 14, 2019
@rswgnu
Copy link

rswgnu commented Apr 27, 2019

So is anyone going to add this? Really need to be able to edit issue titles. Thanks.

@BasThomas
Copy link
Collaborator

Well, feel free to help us push this forward :-)

In the meantime, you can edit titles via the GitHub web interface.

@Huddie
Copy link
Collaborator

Huddie commented Apr 28, 2019

I think the only thing holding this up atm is just the merge conflict. Brian wasn’t sure why one file was showing a weird number of changes but I don’t think there’s an issue there. Should be fine once reviewed and conflicts are resolved

Sent with GitHawk

@rswgnu
Copy link

rswgnu commented Apr 28, 2019 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💤 awaiting review Pull Request is awaiting code reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants