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

[WIP] Merge Fetch15 branch #256

Closed
wants to merge 105 commits into from
Closed

Conversation

@knorth55
Copy link
Member Author

knorth55 commented Aug 18, 2022

Because of conflict, I removed #296 and #323 temporary.
@mqcmd196 please make a goal and complete #296 and #323.
if you want to do new thing, it is better to make a new PR based on #323, not appending #323.

EDIT:
I resolved the conflicts and merged #323.
#296 is not yet merged.

@knorth55
Copy link
Member Author

knorth55 commented Sep 1, 2022

@tkmtnt7000
is this commit included in some PR? if not, please open a PR.
jsk-ros-pkg/jsk_3rdparty@8584b81 (#256)

@tkmtnt7000
Copy link
Member

Thank you. I made PR #378

add actionmsg, catkin install, ros action

Add action server for google chat ros

rename action msg, edit more info to readme

add figs to readme and fix style

Add roseus client

add euslisp sample to readme

add link for jsk internal users

use dummy spacename

Add some comments and fix typo

key args to euslisp

fix with flake8

fix README as review comment

set default type name

fix CMake install directory

rename action name
add space activity msg

fix message names, add more units

add rosparam in launch

using request, not flask

fix so many bugs and works fine with dm, still bugs with space

supress SSL reset error and add header debug

fix unicode bugs, set default dic value
Add and fix message type for supporting card type

support card type, TODO: uploader
fix upload service

fix name bug, remove upload parent dir
fix botton key value json bug

fix value error when get card activity and parse key value
@knorth55
Copy link
Member Author

knorth55 commented Sep 30, 2022

this PR is moved to #383

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

Successfully merging this pull request may close these issues.

7 participants